Repository navigation
CI: analyzers bump PR runs the builds and merges itself when green - #812
Merged
Merged
Conversation
The bump PR was opened with GITHUB_TOKEN, which never starts other workflows, so the required builds never ran and every bump had to be merged by hand with the admin override. Open it with CLASSIC_PAT and turn on auto-merge. A bump that changes the parse-en-us baselines stays open with a red check. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
--recursive moved analyzers' own submodules (nlp-tutorials, nlpfix-analyzers) to their latest commits, so analyzers looked modified even when its pointer hadn't moved, and the commit step failed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The analyzers bump PR (#809 and earlier) was opened with
GITHUB_TOKEN. PRs opened that way never start other workflows, so the required builds never ran and every bump sat blocked until merged by hand with the admin override.CLASSIC_PATsecret, so the builds run on it.final.tree/out.xmlbaseline check and stays open; update the baseline on the PR branch and it merges.The repo setting "Allow auto-merge" is now on (it was off). Merging a bump still does not cut an engine release.
After this merges: close #809 and run the workflow (Actions → Update analyzers submodule → Run workflow) to reopen it the new way.
🤖 Generated with Claude Code