Skip to content

Conversation

@jmeridth
Copy link
Member

Update README noting this new requirement

@jmeridth jmeridth self-assigned this Sep 18, 2025
@jmeridth jmeridth requested a review from a team as a code owner September 18, 2025 04:15
Copilot AI review requested due to automatic review settings September 18, 2025 04:15
Copy link

Copilot AI 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 Overview

This PR updates the release workflow to require both a release label and a semver label (major, minor, or patch) for version bumps, preventing accidental releases from automated PRs like Dependabot.

  • Modifies the release condition logic to require explicit release + semver label combinations
  • Updates documentation to reflect the new dual-label requirement
  • Replaces the previous warning about Dependabot conflicts with clear usage instructions

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
README.md Updates documentation to explain the new dual-label requirement for releases
.github/workflows/release.yaml Modifies release conditions to require both release and semver labels together

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Update README noting this new requirement

Signed-off-by: jmeridth <[email protected]>
@jmeridth jmeridth force-pushed the jm_ensure_semver_labels_are_used_with_release_label branch from ad3f64f to b713cd2 Compare September 18, 2025 04:16
@jmeridth jmeridth changed the title fix: ensure release label is used with semvar labels fix: ensure release label is used with semver labels Sep 18, 2025
@jmeridth jmeridth merged commit 8cd8cf1 into main Sep 18, 2025
11 checks passed
@jmeridth jmeridth deleted the jm_ensure_semver_labels_are_used_with_release_label branch September 18, 2025 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants