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
(maint) don't validate package version as semver in status service #1855
(maint) don't validate package version as semver in status service #1855
Conversation
@@ -78,7 +78,8 @@ | |||
[puppetlabs/trapperkeeper ~tk-version] | |||
[puppetlabs/trapperkeeper-webserver-jetty9 ~tk-jetty9-version] | |||
[prismatic/schema "1.0.4"] | |||
[puppetlabs/trapperkeeper-status ~tk-status-version :exclusions [trptcolin/versioneer]] | |||
[puppetlabs/trapperkeeper-status ~tk-status-version] | |||
[trptcolin/versioneer "0.2.0"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
might want to put this above the status service since it's a dependency of status
@cprice404 does this seem reasonable or should we be fixing the status service to accept SNAPSHOTs? |
d858350
to
61c85de
Compare
@@ -93,6 +94,16 @@ | |||
true | |||
@maint-mode-atom)))) | |||
|
|||
(defn get-artifact-version |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd make a note in a docstring about how this needs to be fixed upstream in clj-semver
This was causing issues in the extensions tests due to our snapshot versioning.
61c85de
to
1da2a17
Compare
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
…e-version-as-semver (maint) don't validate package version as semver in status service
This was causing issues in the extensions tests due to our snapshot versioning.