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 hint #3529

Merged
merged 1 commit into from
Oct 28, 2023
Merged

Fix grammar in currency-exchange hint #3529

merged 1 commit into from
Oct 28, 2023

Conversation

kytrinyx
Copy link
Member

This is a tiny thing, and it probably wouldn't confuse people who are
actually doing the exercise.

I came across it because I was doing a full audit of instances where
we say 'amount of ...' since there were several instances where it
should have been 'number of ...'.

At first I thought this was supposed to be 'number of changes', but
then in the context of the exercise it turns out that it was a
different problem entirely :)

This is a tiny thing, and it probably wouldn't confuse people who are
actually doing the exercise.

I came across it because I was doing a full audit of instances where
we say 'amount of ...' since there were several instances where it
should have been 'number of ...'.

At first I thought this was supposed to be 'number of changes', but
then in the context of the exercise it turns out that it was a
different problem entirely :)
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!

@BethanyG BethanyG added the x:rep/small Small amount of reputation label Oct 28, 2023
@BethanyG BethanyG merged commit 939b0e3 into main Oct 28, 2023
9 checks passed
@BethanyG BethanyG deleted the incorrect-plural branch October 28, 2023 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x:rep/small Small amount of reputation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants