-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Change __version__ on release #529
Conversation
e1978cf
to
ca638b8
Compare
ca638b8
to
53ea5b9
Compare
@@ -15,7 +15,7 @@ | |||
"""Meta data of line-bot-sdk.""" | |||
|
|||
|
|||
__version__ = '3.5.0' | |||
__version__ = "__LINE_BOT_SDK_PYTHON_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.
(check point)
python generate-code.py | ||
- name: Update version in linebot/__about__.py | ||
run: | | ||
sed -i 's/__version__ = "__LINE_BOT_SDK_PYTHON_VERSION__"/__version__ = "12.34.5"/g' linebot/__about__.py |
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.
What's 12.34.5
?
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 should have written why this uses 12.34.5
...
Tox (test library) rejects running test if version is illegal. As a workaround, this embeds dummy 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.
LGTM
#526
To make release easier, this pull request
__version__
defined inlinebot/__about__.py
.__verison__
with${{ github.event.release.tag_name }}
before uploading python package on release. (ref: line-bot-java-sdk)If this change is merged, all we have to do can be