Repository navigation
build, check: report a missing playbook file in check, refuse an unparsed playbook bullet, and reword the drift message - #177
Merged
Merged
Conversation
…k.py, and the drift review line
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.
Follow-ups to the playbook resolver and the upstream merge helper.
scripts/build.pyreports override drift as<rel>: upstream changed since this override was written; review or re-port it, then --update-lock. The module docstring says the same.scripts/check.pynow reports a poteto-mode playbook bullet whose file does not exist.scripts/build.pystill fails on that finding before it swaps the tree, and no longer does the lookup itself.scripts/catalog.pyreads only the Playbooks section, from the line after the level-two headingPlaybooksto the next level-two heading.- **Title.** summaryis a finding and fails the check and the build.AGENTS.mdstep 5 names the missing playbook file case, and the--mergeparagraph names--conflicts <dir>.A tree built from this branch is byte-identical to one built from main.
Reviewed at 6af6cd5b.