-
Notifications
You must be signed in to change notification settings - Fork 390
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
Automatically update package.json version from setup.py #1692
Comments
Would this mean we need to publish a version to PyPI each time we publish to npm? |
If we want automated releases (#1666) then yes.... unless we can have some logic to conditionally publish one package but not the other? |
IMO the breaking changes in these two packages might differ, so I think it's probably ok to version these separately? |
We have adopted use pf tbump quite systematically with success and positive feedback, instead of writing code in setup.py, i suggest we hardcode it and let tbump update it in all locations we have the string. For practical reasons I prefer to see only have one version to track for all parts in this repo, even if it could make sense to let a js client drift. |
Given there seems to be consensus on this, I'm happy to go with it especially as other people are doing the work :D :D |
I think we should keep versioneer (or some other automated versioning tool) because we're installing dev versions of BinderHub in production. |
Proposed change
An additional version string was added in #1689:
binderhub/js/packages/binderhub-client/package.json
Line 3 in 9e2e544
Can we link this up to
setup.py
so that the version is automatically updated to match the BinderHub version?Alternative options
Do nothing
Who would use this feature?
It would ensure all the npm package version matches the BinderHub version
The text was updated successfully, but these errors were encountered: