Repository navigation
upstream PR lane: the crossing workflows keep upstream's triggers, and compare_query_hashes.py crosses with them (#217) - #218
Merged
Conversation
…examples workflow, the crossing workflows keep upstream's trigger block (#217) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Ubz3m6QXHESerCkehC1qgi
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.
Fixes #217.
Two defects in the generated upstream pull request branch. The examples workflow runs
scripts/compare_query_hashes.pyafter example 10, butexamples/scriptsis excluded, so on upstream's CI that step would fail in every example job; the generator re-includes that one file. And our copies of the two crossing workflows differ from upstream's in the trigger block for fork-only reasons (push onmainonly, noworkflow_runon aReleaseworkflow), with a comment naming fork tooling; the generator now splices upstream's trigger block in front of the rest of our file and stops if the marker comment is missing.Checked in the post-release sync clone with
--no-push: 105 files, +9,977/-1,931; the trigger block of both workflows is identical toupstream-main's; the only non-bindings/pythonpaths are the two workflows.🤖 Generated with Claude Code
https://claude.ai/code/session_01Ubz3m6QXHESerCkehC1qgi