Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix grammar in currency-exchange exercise #3517

Merged
merged 1 commit into from
Oct 18, 2023
Merged

Fix grammar in currency-exchange exercise #3517

merged 1 commit into from
Oct 18, 2023

Conversation

kytrinyx
Copy link
Member

@kytrinyx kytrinyx commented Oct 18, 2023

The word amount is generally used for things that come in piles (like money) or
that can otherwise be measured but not counted (like water).

The word number is typically used if you can count the things.

Technically money can be counted, but it's an exception (see 'piles' above).

Here we count bills and digits, so 'number' is the more common choice of word.

The word _amount_ is generally used for things that come in piles (like money) or
that can otherwise be measured but not counted (like water).

The word _number_ is typically used if you can count them.

Technically money can be counted, but it's an exception (see 'piles' above).

Here we count bills and digits, so 'number' is the more common choice of word.
Copy link
Member

@BethanyG BethanyG left a comment

Choose a reason for hiding this comment

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

Thank you for this! 🌟 🚀

@BethanyG BethanyG merged commit ad890c1 into main Oct 18, 2023
8 checks passed
@BethanyG BethanyG deleted the amount-vs-number branch October 18, 2023 11:12
kytrinyx added a commit that referenced this pull request Oct 26, 2023
This is a follow-up to #3517

I've fixed the following occurrences:
- number of values
- number of digits
- number of function calls
- number of hours
- number of health points
- number of (intermediate) delivery points
- number of keyword arguments
- number of grains
- number of names

I have also verified that the following are correct:
- amount of reputation
- amount of work
- amount of functionality
- amount of memory
- amount of money
- amount of domestic currency
- amount of tree climbing
- amount of energy
BethanyG pushed a commit that referenced this pull request Oct 27, 2023
This is a follow-up to #3517

I've fixed the following occurrences:
- number of values
- number of digits
- number of function calls
- number of hours
- number of health points
- number of (intermediate) delivery points
- number of keyword arguments
- number of grains
- number of names

I have also verified that the following are correct:
- amount of reputation
- amount of work
- amount of functionality
- amount of memory
- amount of money
- amount of domestic currency
- amount of tree climbing
- amount of energy

[no important files changed]
hussammousa68 pushed a commit to hussammousa68/test that referenced this pull request Oct 13, 2024
This is a follow-up to exercism/python#3517

I've fixed the following occurrences:
- number of values
- number of digits
- number of function calls
- number of hours
- number of health points
- number of (intermediate) delivery points
- number of keyword arguments
- number of grains
- number of names

I have also verified that the following are correct:
- amount of reputation
- amount of work
- amount of functionality
- amount of memory
- amount of money
- amount of domestic currency
- amount of tree climbing
- amount of energy

[no important files changed]
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