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

Integrate automated PyPI releases into CI #125

Merged
merged 4 commits into from
Apr 23, 2024
Merged

Conversation

vdusek
Copy link
Collaborator

@vdusek vdusek commented Apr 23, 2024

@github-actions github-actions bot added this to the 88th sprint - Tooling team milestone Apr 23, 2024
@github-actions github-actions bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Apr 23, 2024
@vdusek vdusek force-pushed the integrate-releases-into-ci branch 2 times, most recently from 183b084 to 8f2d1c9 Compare April 23, 2024 10:59
@vdusek vdusek marked this pull request as ready for review April 23, 2024 11:00
- name: Determine release type
id: determine-release-type
run: |
if [[ ${{ github.event_name }} = release ]]; then
Copy link
Member

Choose a reason for hiding this comment

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

this is fine for now, but in the long run i would like to see similar mechanism as we have in crawlee - stable releases are triggered via workflow dispatch, and they modify the changelog as well as create the git tag and its corresponding GH release notes. as if you would trigger this by creating a GH release first, it means you are responsible for creating the changelog - i know there is button in the UI for that nowadays, but it ignores commits to master and doesn't handle conventional commits properly (e.g. it adds everything including chore commits)

Copy link
Collaborator

Choose a reason for hiding this comment

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

AFAIK this is copied from python SDK, do we want to make those changes in both repos?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

will be done later as part of #18

Copy link
Member

@B4nan B4nan Apr 23, 2024

Choose a reason for hiding this comment

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

Yes, ideally, we will migrate all repositories to the new system, but we agreed with Vlada to keep this as is for now and deal with this later (even after the public launch, functionality has priority over this). Also @vladfrangu should prepare a new mechanism for releasing the CLI which might help here too, as it should do the same, workflow dispatch will trigger changelog update and create new git tag and gh release, and this part should work the same regardless of the language.

Copy link
Collaborator

@janbuchar janbuchar left a comment

Choose a reason for hiding this comment

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

Well, it looks like it can hold water. Let's see 🙂

@vdusek vdusek merged commit 08fe6a0 into master Apr 23, 2024
19 checks passed
@vdusek vdusek deleted the integrate-releases-into-ci branch April 23, 2024 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t-tooling Issues with this label are in the ownership of the tooling team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integrate automated PyPI releases into CI
3 participants