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

min, not minder. Minder means less. #172

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

picobyte
Copy link

@picobyte picobyte commented Feb 21, 2023

minder means: less, -1 should be worded as min één or minus één, I'd suggest the former, the latter is a bit more formal, used to indicate a correction is due: off-by-one. The first test should also be adapted, similarly.

Also in German minus there seems a trailing space is added. Not sure if that is a problem

For reference:
https://nl-m-wikipedia-org.translate.goog/wiki/Minteken?_x_tr_sl=auto&_x_tr_tl=en&_x_tr_hl=nl&_x_tr_pto=wapp
see the second dot below use / gebruik (in the dutch explanation)

`minder' means: less, -1 should be worded as `min één' or `minus één', I'd suggest the latter, the former is a bit more formal, used to indicate a correction is due: off-by-one.
The first test should also be adapted, similarly.
@picobyte picobyte requested a review from moul as a code owner February 21, 2023 18:25
@trafico-bot trafico-bot bot added the 🔍 Ready for Review Pull Request is not reviewed yet label Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔍 Ready for Review Pull Request is not reviewed yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant