Skip to content

Add "Special note for JVM"#180

Closed
Primetalk wants to merge 3 commits intosemver:gh-pagesfrom
Primetalk:patch-1
Closed

Add "Special note for JVM"#180
Primetalk wants to merge 3 commits intosemver:gh-pagesfrom
Primetalk:patch-1

Conversation

@Primetalk
Copy link
Copy Markdown

ref #341

a bug or introduce new functionality. I would usually expect additional code
for the latter instance, in which case it's obviously a minor level increment.

#### Special note for JVM or similar environments
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The FAQ is currently part of the Semver spec itself, thus this should go in https://github.com/semver/semver/blob/master/semver.md. When the next version of Semver is finalized, it'll get copied over here and translated.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a PR semver/semver#414 and a comment semver/semver#414 (review)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@haacked There is an opposite opinion - semver/semver#414 (comment). Could you discuss with @jwdonahue ?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Primetalk, @haacked is the authority on this, my apologies for any confusion.

index.md Outdated

In such environments where only one version of a library could
be present in a running system, the transitive dependencies effectively become
part of the public library interface and thus the version of the user library SHOULD be
Copy link
Copy Markdown

@fverdino fverdino Dec 13, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not reword section in order to use MUST (instead of SHOULD) ?

It would be better to clarify when you MUST do something. Instead of simply recommending with SHOULD.

Indeed, the potential effect is that libraries will still be semantic-versioning compliant even if they do not follow at all your recommendation!

I suggest to add something like "[...] If any potential client environment allows only one version of a library be present in running system, [...] user library MUST be [...]"

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated

In linearized environments where all dependencies are resolved to just a single version, we MUST increment the version of the library to the same extent as transitive dependencies change.
index.md Outdated
#### Special note for JVM or similar environments

In such environments where only one version of a library could
I suggest to add something like "[...] If any potential client environment allows only one version of a library be present in running system, [...] user library MUST be [...]"
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess line 229 (starting with "I suggest ...") was added in your last commit for mistake, it should be removed.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I'm sorry for the inconvenience.

@haacked
Copy link
Copy Markdown
Contributor

haacked commented Nov 24, 2018

Closing this until it's resolved on SemVer proper.

@haacked haacked closed this Nov 24, 2018
@jwdonahue
Copy link
Copy Markdown

@haacked, I thought we had resolved to do something like this in #414. Does this need to be a PR in semver/semver instead of here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants