Skip to content

chore: factor release note checking into separate script#3628

Open
rickeylev wants to merge 1 commit intobazel-contrib:mainfrom
rickeylev:chore.split.version.marker.check
Open

chore: factor release note checking into separate script#3628
rickeylev wants to merge 1 commit intobazel-contrib:mainfrom
rickeylev:chore.split.version.marker.check

Conversation

@rickeylev
Copy link
Collaborator

This is to make it easier to check for missing version markers by running the script directly

@rickeylev rickeylev requested a review from aignas as a code owner February 23, 2026 01:24
@gemini-code-assist
Copy link
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@rickeylev rickeylev enabled auto-merge February 23, 2026 01:24
@rickeylev rickeylev force-pushed the chore.split.version.marker.check branch from 241bea7 to 3ed9c0d Compare February 23, 2026 01:24
# Update our local state so the grep command below searches what we expect
git checkout "$TAG"

# Exclude dot directories, specifically, this file so that we don't
Copy link
Collaborator

Choose a reason for hiding this comment

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

[Optional, non-blocking] It might be prudent to call check_version_markers.sh from this script.

  • Removes the extra step in the release flow (I'm not sure what the GH step overhead is)
  • There's no risk of forgetting to check for VERSION_NEXT markers (if this create_archive_and_notes.sh` script is ever called outside of CI).

@rickeylev rickeylev added this pull request to the merge queue Feb 23, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants