-
-
Notifications
You must be signed in to change notification settings - Fork 76
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
Switch versioning system to use autover #143
Conversation
I have released 1.5.0a1 here using this PR (and pushed the tag). As far as I can tell, the tests are only failing because of a few flake issues. |
@philippjfr I think this is ready to review/merge when the tests pass and when you confirm that the alpha works as expected. |
I think this PR is now ready to review/merge. |
@@ -0,0 +1,2 @@ | |||
__init__.py export-subst | |||
setup.py export-subst |
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.
Newlines here and in .gitignore would be nice, but won't let it hold up the PR.
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.
Addressed in 3efb09f
One minor comment about newlines, I'll let you decide what to do, happy to merge either way. |
Addressed your comment and the tests are green. |
PR to switch geoviews to autover, following the corresponding PR in holoviz/holoviews#2426. Embedding version.py until it is available via param.
The only thing I'm not entirely sure about is whether a MANIFEST will need to be added for the
.version
file.