Skip to content

ci: Add an action to release the library #916

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

Merged
merged 1 commit into from
Jan 25, 2021
Merged

Conversation

jan-auer
Copy link
Member

Adds another action similar to the main Release action that publishes the Python
library. It runs craft in the py subfolder.

#skip-changelog

@jan-auer jan-auer requested a review from a team January 25, 2021 08:34
@jan-auer jan-auer requested a review from a team as a code owner January 25, 2021 08:34
with:
version: ${{ github.event.inputs.version }}
force: ${{ github.event.inputs.force }}
path: py
Copy link
Member

Choose a reason for hiding this comment

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

Why not parametrize the main release action instead?

Copy link
Member Author

Choose a reason for hiding this comment

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

I feared that this would be too error prone. One is CalVer, the other one requires a version. One is released on schedule, the other only ever manually.

They do look quite similar, though.

Copy link
Member

Choose a reason for hiding this comment

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

Hmm, we can set CalVer automatically and disable if path exists but I'm not sure if the complexity is worth it at this point. Maybe we can have the duplication until we have more then?

Copy link
Member Author

Choose a reason for hiding this comment

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

Sounds good to me, let's go ahead with the current patch then. Thanks!

@jan-auer jan-auer merged commit 1502e6e into master Jan 25, 2021
@jan-auer jan-auer deleted the ci/release-library branch January 25, 2021 08:58
jan-auer added a commit that referenced this pull request Jan 27, 2021
* master:
  fix: Build proper 64-bit Linux wheel
  release: 0.8.2
  chore: Switch to manylinux2010 (#917)
  meta(py): Changelog for 0.8.2
  ci: Add an action to release the library (#916)
  ref: Remove Compatibility mode for Session Aggregates (#913)
  release: 21.1.0
  ci: Fix clippy command invocation
  ci: Run clippy via the cargo action
  feat(server): Ingest and normalize sample rates (#910)
jan-auer added a commit that referenced this pull request Jan 28, 2021
* master:
  fix(server): Log when recovering from network outages (#918)
  feat: Add a .no-cache option to event sending (#911)
  fix: Build proper 64-bit Linux wheel
  release: 0.8.2
  chore: Switch to manylinux2010 (#917)
  meta(py): Changelog for 0.8.2
  ci: Add an action to release the library (#916)
  ref: Remove Compatibility mode for Session Aggregates (#913)
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.

3 participants