Skip to content
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

Publish to (Test)PyPI @ GHA using pypi-publish #471

Merged
merged 2 commits into from
Mar 27, 2023

Conversation

webknjaz
Copy link
Contributor

This patch switches away from using hatch publish to the PyPA GitHub action that performs distribution publishing using Twine.

Resolves: #465.

This patch switches away from using `hatch publish` to the PyPA GitHub
action that performs distribution publishing using Twine.

Resolves: jorisroovers#465.
@coveralls
Copy link

coveralls commented Mar 11, 2023

Coverage Status

Coverage: 99.662%. Remained the same when pulling ccd0b93 on webknjaz:maintenance/pypi-publish into acc9d9d on jorisroovers:main.

- name: Publish gitlint 🐍📦 to PyPI
uses: pypa/gh-action-pypi-publish@release/v1
with:
packages-dir: gitlint-core/dist/
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
packages-dir: gitlint-core/dist/

I think this should be the top-level dist/ package, which according to the docs is the default value for packages-dir.

- name: Publish gitlint-core 🐍📦 to TestPyPI
uses: pypa/gh-action-pypi-publish@release/v1
with:
password: ${{ secrets.TEST_PYPI_GITLINT_CORE_PASSWORD }}
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
password: ${{ secrets.TEST_PYPI_GITLINT_CORE_PASSWORD }}
password: ${{ secrets.TEST_PYPI_GITLINT_CORE_PASSWORD }}
packages-dir: gitlint-core/dist/

@jorisroovers jorisroovers merged commit a3ea8ae into jorisroovers:main Mar 27, 2023
@jorisroovers jorisroovers added this to the 0.20.0 milestone Mar 27, 2023
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.

Consider adopting pypi-publish GHA instead of hatch publish
3 participants