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

Add workflow with building pip #344

Merged
merged 6 commits into from
Jan 27, 2025
Merged

Add workflow with building pip #344

merged 6 commits into from
Jan 27, 2025

Conversation

pawloch00
Copy link
Collaborator

@pawloch00 pawloch00 commented Jan 27, 2025

Fixes / Features

  • Add workflow to upload to pip repository

Testing / Documentation

Testing details.

  • [ y/n ] Tests pass
  • [ y/n ] Appropriate changes to documentation are included in the PR

@pawloch00 pawloch00 marked this pull request as ready for review January 27, 2025 12:28
Signed-off-by: Piotr Pawłowski <[email protected]>
uses: pypa/gh-action-pypi-publish@release/v1
with:
repository-url: https://test.pypi.org/legacy/

Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: Remove empty line

with:
name: python-package-distributions
path: dist/
- name: Publish distribution 📦 to TestPyPI
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: Inconsistent naming of steps between publish-to-testpypi and publish-to-pypi.

repository-url: https://test.pypi.org/legacy/

publish-to-pypi:
name: >-
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: Why we need this line break with ">-"?

44past4
44past4 previously approved these changes Jan 27, 2025
Signed-off-by: Piotr Pawłowski <[email protected]>
@pawloch00 pawloch00 merged commit 5259676 into main Jan 27, 2025
6 of 7 checks passed
@pawloch00 pawloch00 deleted the ppawl-build-wheels branch January 27, 2025 12:51
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.

2 participants