Skip to content

Comment improvements - #132

Open
dambem wants to merge 2 commits into
masterfrom
docs
Open

Comment improvements#132
dambem wants to merge 2 commits into
masterfrom
docs

Conversation

@dambem

@dambem dambem commented Aug 8, 2020

Copy link
Copy Markdown
Owner

just a PR to add some more comments and hopefully make some of the functions more clear!

@Megan-Wright Megan-Wright 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.

Couple of tiny little things but otherwise looks good 👍

/**
* Human display, append HTML with showing how many people would lose they're lives due to pollution
* @param {int} people - Amount of people
* @param {int} infected - Amount of people

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.

Amount of people infected?

}

/**
* Human display, append HTML with showing how many people would lose they're lives due to pollution

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.

*their lives

@dambem

dambem commented Aug 9, 2020

Copy link
Copy Markdown
Owner Author

Once my other PR is through i'll update that fully and get the changes sorted!

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