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 package #50

Merged
merged 34 commits into from
Oct 27, 2023
Merged

Publish package #50

merged 34 commits into from
Oct 27, 2023

Conversation

fnberta
Copy link
Contributor

@fnberta fnberta commented Oct 25, 2023

The idea for publishing is that the CHANGELOG.md file is the source of truth for the version number. To make a new release, you add a new version entry in the changelog and push and merge this change. A github action will detect a new version number and build and publish a new package. The action also adds a git tag and github release reflecting the new version.

However, there is some logic needed for this and we want a way to publish until this is in a good shape. This PR adds a simple workflow that listens to PRs being merged to main whose head branch is release/<version-number>. It uses the version-number part to version the package.

This allows us to read the version from the current git tag instead of having it hardcoded in some file.
@fnberta fnberta self-assigned this Oct 25, 2023
@fnberta fnberta marked this pull request as ready for review October 25, 2023 21:24
@fnberta fnberta requested a review from a team as a code owner October 25, 2023 21:24
It doesn't look like it's used anywhere and breaks publishing to PIPY.
It's not needed anymore since we only listen to `workflow_dispatch` events.
Copy link
Contributor

@aiAdrian aiAdrian left a comment

Choose a reason for hiding this comment

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

Well done!

@fnberta fnberta merged commit aa993a1 into main Oct 27, 2023
6 checks passed
@fnberta fnberta deleted the publish branch October 27, 2023 12:58
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