Skip to content

High score#12

Open
sangeethailango wants to merge 3 commits into
mainfrom
high-score
Open

High score#12
sangeethailango wants to merge 3 commits into
mainfrom
high-score

Conversation

@sangeethailango

Copy link
Copy Markdown
Owner

No description provided.

@sangeethailango sangeethailango self-assigned this Jun 8, 2023
Comment thread high-score/lib/high_score.ex Outdated
%{}
end

def add_player(scores, name, score \\ @default_initial_score ) do

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
def add_player(scores, name, score \\ @default_initial_score ) do
def add_player(scores, name, score \\ @default_initial_score) do

Remove the empty space

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@nittin-shankar nittin-shankar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job @sangeethailango. Just a small correction and you're good to go!

Make sure you watch other people's solutions for every problem you solve

@nittin-shankar nittin-shankar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but just make sure you understand the tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants