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

CI: Add release workflow for staging the Python package through GHA to PyPI #41

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

amotl
Copy link
Member

@amotl amotl commented Jun 17, 2024

About

A release workflow for staging the Python package through GHA to PyPI.
Along the lines, the patch includes "developer guide" documentation for both Python and JavaScript.

Along the lines, the patch includes "developer guide" documentation for
both Python and JavaScript.
Comment on lines +63 to +69
Run specific tests:

???

Format code:

???
Copy link
Member Author

Choose a reason for hiding this comment

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

Didn't know relevant incantations for the JavaScript domain. Are there any?

Copy link
Collaborator

@surister surister Jun 18, 2024

Choose a reason for hiding this comment

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

npm test and npm build

But I can add them later in the upcoming javascript patch

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah sorry I didn't see the "?", no currently there is no format, maybe we should add it int he future with prettier, for running a single test is npm test -- other.test.js

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks. Yeah, please fill in the gaps, or delete sections where applicable.

Comment on lines 30 to 36
- name: Publish package to PyPI
uses: pypa/gh-action-pypi-publish@release/v1
with:
user: __token__
password: ${{ secrets.PYPI_API_TOKEN }}
Copy link
Member Author

Choose a reason for hiding this comment

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

The GHA workflow needs the PYPI_API_TOKEN secret, which has just been added to the repository.

@amotl amotl merged commit 6a98ab1 into main Jun 18, 2024
8 checks passed
@amotl amotl deleted the amo/ci-release branch June 18, 2024 09:53
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