[GH-ISSUE #292] How to handle non-api changes? #6201

Closed
opened 2026-06-17 04:51:17 -05:00 by GiteaMirror · 5 comments
Owner

Originally created by @Lucretiel on GitHub (Feb 7, 2016).
Original GitHub issue: https://github.com/semver/semver/issues/292

Let's say I merge a number of pull requests that add a bunch of test coverage, introduce commenting, or update the documentation that ships with the project. How should I handle the version number in these cases? Any or all of these changes could be part of a "release," as they're downloaded with the package. The former case is interesting because it affect people who run the tests, but not the typical client of the library. Are any of these considered "bug fixes," if nothing about the actual usage or API of the project changes? My instinct is to simply revise an existing version (prohibited by rule 3). Would this be a patch revision? Build metadata?

Originally created by @Lucretiel on GitHub (Feb 7, 2016). Original GitHub issue: https://github.com/semver/semver/issues/292 Let's say I merge a number of pull requests that add a bunch of test coverage, introduce commenting, or update the documentation that ships with the project. How should I handle the version number in these cases? Any or all of these changes could be part of a "release," as they're downloaded with the package. The former case is interesting because it affect people who run the tests, but not the typical client of the library. Are any of these considered "bug fixes," if nothing about the actual usage or API of the project changes? My instinct is to simply revise an existing version (prohibited by [rule 3](http://semver.org/#spec-item-3)). Would this be a patch revision? Build metadata?
Author
Owner

@FichteFoll commented on GitHub (Feb 7, 2016):

Imo either minor or patch, depending on how tests are part of the API, favoring patch.

<!-- gh-comment-id:180984415 --> @FichteFoll commented on GitHub (Feb 7, 2016): Imo either minor or patch, depending on how tests are _part of the API_, favoring patch.
Author
Owner

@PowerKiKi commented on GitHub (Feb 8, 2016):

I'd go for patch too

<!-- gh-comment-id:181446808 --> @PowerKiKi commented on GitHub (Feb 8, 2016): I'd go for patch too
Author
Owner

@crazedsanity commented on GitHub (Feb 8, 2016):

I increment the patch level in these cases.

<!-- gh-comment-id:181521716 --> @crazedsanity commented on GitHub (Feb 8, 2016): I increment the patch level in these cases.
Author
Owner

@manbusky commented on GitHub (Mar 12, 2016):

Such as performance optimization, I will increase the patch.

<!-- gh-comment-id:195760245 --> @manbusky commented on GitHub (Mar 12, 2016): Such as performance optimization, I will increase the patch.
Author
Owner

@jwdonahue commented on GitHub (Dec 1, 2017):

@Lucretiel, unless you have any further questions on this matter, please close this issue.

<!-- gh-comment-id:348367294 --> @jwdonahue commented on GitHub (Dec 1, 2017): @Lucretiel, unless you have any further questions on this matter, please close this issue.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/semver#6201