Skip to content

Conversation

@Tom-Willemsen
Copy link
Member

@Tom-Willemsen Tom-Willemsen commented Jul 24, 2025

Description of work

Add a check to ensure PRs can't be merged without a release label.

Ticket

None

Labels

Add appropriate label(s) to this PR

  • 'bluesky-Semver-Major' - Breaking Change
  • 'bluesky-Semver-Minor' - New feature
  • 'bluesky-bug' - Bug fix
  • 'bluesky-documentation' - Document Changes
  • 'bluesky-ignore-for-release' - Do not show in the release notes

bit meta ;)

Acceptance criteria

  • Pull request title is understandable for a user (e.g. scientist) reading the release notes. The PR title should be a short description of the change from a user perspective.
  • Pull request has appropriate labels for automatic release-notes generation
  • Check fails if no appropriate release-notes labels on the PR
  • Check passes if a release note label is on the PR

Documentation

Highlight and provide a link to any additions or changes to the documentation, if applicable. The aim is provide information to help the reviewer

@Tom-Willemsen Tom-Willemsen added the bluesky-Semver-Major Breaking changes label Jul 24, 2025
@Tom-Willemsen Tom-Willemsen force-pushed the plz_add_labels_to_prs branch from 5926dd5 to 93cb24a Compare July 24, 2025 10:44
@Tom-Willemsen Tom-Willemsen added bluesky-ignore-for-release Non user-facing changes (e.g. CI setup) and removed bluesky-Semver-Major Breaking changes labels Jul 24, 2025
@Tom-Willemsen Tom-Willemsen marked this pull request as ready for review July 24, 2025 10:46
@Tom-Willemsen Tom-Willemsen requested a review from rerpha July 24, 2025 10:46
@Tom-Willemsen Tom-Willemsen changed the title Add labels plz Enforce adding release-note labels to PRs Jul 24, 2025
@rerpha rerpha merged commit f6f5001 into main Jul 24, 2025
15 of 16 checks passed
@rerpha rerpha deleted the plz_add_labels_to_prs branch July 24, 2025 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bluesky-ignore-for-release Non user-facing changes (e.g. CI setup) size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants