Inconsistent language about build metadata and precedence #97

Closed
opened 2026-02-17 11:13:25 -06:00 by GiteaMirror · 5 comments
Owner

Originally created by @toddreed on GitHub (Aug 8, 2013).

Section 10 states (emphasis added):

Build metadata SHOULD be ignored when determining version precedence.

And Section 11 states:

Build metadata does not figure into precedence.

For these statements to have greater consistency, I think Section 10 should state:

Build metadata MUST be ignored when determining version precedence.

Originally created by @toddreed on GitHub (Aug 8, 2013). Section 10 states (emphasis added): > Build metadata **SHOULD** be ignored when determining version precedence. And Section 11 states: > Build metadata **does not** figure into precedence. For these statements to have greater consistency, I think Section 10 should state: > Build metadata **MUST** be ignored when determining version precedence.
Author
Owner

@haacked commented on GitHub (Aug 8, 2013):

Yeah, I agree. Send a PR?

@haacked commented on GitHub (Aug 8, 2013): Yeah, I agree. Send a PR?
Author
Owner

@pvdb commented on GitHub (Aug 21, 2013):

Does merging #151 mean this ticket can be closed?

@pvdb commented on GitHub (Aug 21, 2013): Does merging #151 mean this ticket can be closed?
Author
Owner

@haacked commented on GitHub (Aug 21, 2013):

Yep!

@haacked commented on GitHub (Aug 21, 2013): Yep!
Author
Owner

@haacked commented on GitHub (Aug 21, 2013):

@toddreed for future reference, if you add "closes #150" Where 150 is the issue number, in your commit message or in the body of the pull request, it'll close the issue automatically when the PR is merged. :)

@haacked commented on GitHub (Aug 21, 2013): @toddreed for future reference, if you add "closes #150" Where 150 is the issue number, in your commit message or in the body of the pull request, it'll close the issue automatically when the PR is merged. :)
Author
Owner

@toddreed commented on GitHub (Aug 21, 2013):

@Haacked Ahh—thanks for the tip.

@toddreed commented on GitHub (Aug 21, 2013): @Haacked Ahh—thanks for the tip.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/semver#97