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: Release process updates #385

Merged
merged 11 commits into from
Aug 6, 2024
Merged

ci: Release process updates #385

merged 11 commits into from
Aug 6, 2024

Conversation

janbuchar
Copy link
Collaborator

@janbuchar janbuchar added the t-tooling Issues with this label are in the ownership of the tooling team. label Jul 31, 2024
@janbuchar janbuchar requested review from vdusek and B4nan July 31, 2024 11:45
@github-actions github-actions bot added this to the 95th sprint - Tooling team milestone Jul 31, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Pull Request Tookit has failed!

Pull request is neither linked to an issue or epic nor labeled as adhoc!

Copy link
Member

@B4nan B4nan left a comment

Choose a reason for hiding this comment

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

maybe we could also render the names as links in the changelog so they work when rendered in the docs?

@B4nan B4nan marked this pull request as draft July 31, 2024 13:52
@janbuchar janbuchar linked an issue Aug 1, 2024 that may be closed by this pull request
7 tasks
@janbuchar janbuchar marked this pull request as ready for review August 1, 2024 21:57
vdusek

This comment was marked as resolved.

@janbuchar
Copy link
Collaborator Author

Good job! ... Just, could you please update .pre-commit-config.yaml as well?

Huh, I just keep forgetting that one...

Copy link
Collaborator

@vdusek vdusek left a comment

Choose a reason for hiding this comment

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

Could you also please update CONTRIBUTING.md regarding the release process?

@janbuchar janbuchar requested a review from vdusek August 6, 2024 11:57
Copy link
Collaborator

@vdusek vdusek left a comment

Choose a reason for hiding this comment

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

lgtm; thanks

@janbuchar janbuchar merged commit bce0271 into master Aug 6, 2024
16 of 20 checks passed
@janbuchar janbuchar deleted the release-process-updates branch August 6, 2024 13:27
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.

Finalize release process
3 participants