Skip to content

Conversation

@iamsobanjaved
Copy link
Contributor

@iamsobanjaved iamsobanjaved commented Aug 17, 2020

@iamsobanjaved iamsobanjaved force-pushed the pypi-release branch 2 times, most recently from 2c07f7e to 8bfe38f Compare August 18, 2020 11:45
uses: pypa/gh-action-pypi-publish@master
with:
user: __token__
password: ${{ secrets.PYPI_UPLOAD_TOKEN }} No newline at end of file
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

file should end with an empty line

@iamsobanjaved iamsobanjaved requested a review from mraarif March 17, 2021 09:37
@iamsobanjaved iamsobanjaved marked this pull request as ready for review April 8, 2021 14:47
Copy link
Member

@kdmccormick kdmccormick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good; thanks @iamsobanjaved .

One request - when you squash & merge, please revise the commit message to a conventional commit message. Something like:

build: release to pypi

add a GitHub action to push release to PyPI.

BOM-1873

add a GitHub action to push release to PyPI.

BOM-1873
@iamsobanjaved iamsobanjaved merged commit 437c936 into master Apr 15, 2021
@iamsobanjaved iamsobanjaved deleted the pypi-release branch April 15, 2021 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants