[GH-ISSUE #665] Suggestion: Rename positive digit in the EBNF ruleset to nonzero digit #2189

Open
opened 2026-04-20 10:11:00 -05:00 by GiteaMirror · 7 comments
Owner

Originally created by @dtonhofer on GitHub (Feb 22, 2021).
Original GitHub issue: https://github.com/semver/semver/issues/665

I suggest to rename the positive digit in the EBNF ruleset to nonzero digit

Because "positive" is not really an adjective applicable to a "digit": it's a digit, not a number.

So we have the digit "zero" expressed by "0" and the other digits, the nonzero digits

Originally created by @dtonhofer on GitHub (Feb 22, 2021). Original GitHub issue: https://github.com/semver/semver/issues/665 I suggest to rename the `positive digit` in the EBNF ruleset to `nonzero digit` Because "positive" is not really an adjective applicable to a "digit": it's a digit, not a number. So we have the digit "zero" expressed by `"0"` and the other digits, the `nonzero digit`s
GiteaMirror added the questionupdate labels 2026-04-20 10:11:00 -05:00
Author
Owner

@ghost commented on GitHub (Feb 26, 2021):

This is pretty easy to fix and mathematically correct. @dtonhofer do you mind putting in a PR?

<!-- gh-comment-id:786353862 --> @ghost commented on GitHub (Feb 26, 2021): This is pretty easy to fix and mathematically correct. @dtonhofer do you mind putting in a PR?
Author
Owner

@dtonhofer commented on GitHub (Feb 26, 2021):

Okay, will do it later today (hopefully).

<!-- gh-comment-id:786553596 --> @dtonhofer commented on GitHub (Feb 26, 2021): Okay, will do it later today (hopefully).
Author
Owner

@dtonhofer commented on GitHub (Feb 28, 2021):

Submitted a pull request.

https://github.com/semver/semver/pull/673

Note the following:

  • I have not upgrade the version at the top of the page, still at 2.0.0. Technically, should become 2.0.1 (?)
  • I have used the name non-zero digit rather than nonzero digit similar to the existing non-digit.It's a matter of taste.
<!-- gh-comment-id:787433491 --> @dtonhofer commented on GitHub (Feb 28, 2021): Submitted a pull request. https://github.com/semver/semver/pull/673 Note the following: - I have not upgrade the version at the top of the page, still at 2.0.0. Technically, should become 2.0.1 (?) - I have used the name `non-zero digit` rather than `nonzero digit` similar to the existing `non-digit`.It's a matter of taste.
Author
Owner

@ghost commented on GitHub (Mar 1, 2021):

Yes, 2.0.1.

<!-- gh-comment-id:787834867 --> @ghost commented on GitHub (Mar 1, 2021): Yes, 2.0.1.
Author
Owner

@dtonhofer commented on GitHub (Mar 1, 2021):

Changed. Seems to have updated the pull request accordingly.

<!-- gh-comment-id:788030776 --> @dtonhofer commented on GitHub (Mar 1, 2021): Changed. Seems to have updated the pull request accordingly.
Author
Owner

@roniesha1994 commented on GitHub (Mar 21, 2022):

<!-- gh-comment-id:1073909245 --> @roniesha1994 commented on GitHub (Mar 21, 2022): >
Author
Owner

@dtonhofer commented on GitHub (Jul 5, 2022):

Good day. What happened to this? It seems to be in limbo. Accept? Reject?

<!-- gh-comment-id:1175139812 --> @dtonhofer commented on GitHub (Jul 5, 2022): Good day. What happened to this? It seems to be in limbo. Accept? Reject?
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/semver#2189