-
-
Notifications
You must be signed in to change notification settings - Fork 350
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
Put version string in __init__.py #214
Conversation
84290fb
to
97fa1cc
Compare
setup.py
Outdated
long_description=get_long_description(), | ||
long_description_content_type="text/markdown", | ||
author="Kentaro Wada", | ||
author_email="www.kentaro.wada@gmail.com", |
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.
Is this actually the email you wan to put here?
You have www.
as a prefix?
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.
yes, because my name is very common in Japan :)
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.
It is a very unique name to me 🤠
setup.py
Outdated
sys.exit(1) | ||
|
||
commands = [ | ||
"git pull origin main", |
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.
This seems to be a local first release process.
- Clone locally.
- Change the version constant.
- Run the setup script.
It can get very tiresome to change that constant locally, but it does help in the sense that when you download the code without git, you can see the version in the source.
Personally I would put these in a separate release script (calling bash from python is tiring to debug).
Other release process would be more github first, where things are triggered solely on the basis of tags.
Ultimately, not using pkg_resources is what I care about, this seems to address that
97fa1cc
to
3dbfe5a
Compare
Close #213