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

don't fail automatic cherry picks for lack of release notes #21908

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

cburroughs
Copy link
Contributor

We explicitly remove release notes when they cause a git conflict, and rely on the GitHub "release" as the effective point release notes.

Unsure how to test, but setting --label multiple times should work per the test in cli/cli@1ff37be.

We explicitly remove release notes when they cause a git conflict, and
rely on the GitHub "release" as the effective point release notes.

Unsure how to test, but setting `--label` multiple times should work
per the test in cli/cli@1ff37be.
@cburroughs cburroughs self-assigned this Feb 3, 2025
@cburroughs cburroughs added the release-notes:not-required PR doesn't require mention in release notes label Feb 3, 2025
Copy link
Contributor

@huonw huonw left a comment

Choose a reason for hiding this comment

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

yeah, this makes sense to me 👍

@huonw
Copy link
Contributor

huonw commented Feb 3, 2025

For testing... just testing "in production" seems reasonable to me, if you're willing to keep an eye on it and iterate as required.

@cburroughs cburroughs merged commit 4065cfc into pantsbuild:main Feb 5, 2025
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-notes:not-required PR doesn't require mention in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants