chore: remove 15 zero-match no_run entries and the dead guides/results override - #37
Merged
Merged
Conversation
…s override
15 of this repo's 16 no_run.yaml entries matched zero files, as did its only
env_vars.yaml override pattern. They were copied in from autogalaxy_workspace:
11 of the dead no_run entries and the `guides/results/` override name files
that exist only in that repo. HowToGalaxy contains only
chapter_*/tutorial_*.py, so a workspace path such as `gui/mask` or
`guides/results/start_here` can never match here — it is silently inert, not
a skip.
Keeps the only live entry, `tutorial_searches`.
`overrides: []` is written explicitly rather than left as a bare key, because
env_config.py reads `.get("overrides", [])` and an empty key loads as None,
which would raise on iteration.
Behaviour-neutral: for all 33 scripts here the resolved should_skip set and
the resolved build_env_for_script dict are identical before and after, since
every removed entry matched zero files.
Closes part of PyAutoLabs/HowToLens#46
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011ZVaiShnRkv1xbQBegGK1N
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
1 participant
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.
Summary
15 of this repo's 16
no_run.yamlentries matched zero files, as did its onlyenv_vars.yamloverride pattern. They were copied in from autogalaxy_workspace: 11 of the deadno_runentries and theguides/results/override name files that exist only in that repo. HowToGalaxy contains onlychapter_*/tutorial_*.py, so a workspace path such asgui/maskorguides/results/start_herecan never match here — it is silently inert, not a skip.Keeps the only live entry,
tutorial_searches.Part of a 5-repo census (PyAutoLabs/HowToLens#46) that found 50 of 122
no_run.yamlentries (41%) dead across the organism, plus 11 deadenv_varspatterns. Root cause is bidirectional copy-paste between the HowTo* repos and their parent workspaces.The cleaned file gains a header recording the matcher rule and that the list is repo-local and must not be copied between repos; the copy-paste is the actual defect, and a purge without it just resets the clock.
overrides: []is written explicitly rather than left as a bare key, becauseenv_config.py:73reads.get("overrides", [])and an empty key loads asNone, which would raise on iteration.Scripts Changed
None — this PR is config-only.
config/build/no_run.yaml— 16 entries → 1 (kepttutorial_searches)config/build/env_vars.yaml— 1 override pattern →overrides: [](guides/results/)Test Plan
should_skipset and the fully-resolvedbuild_env_for_scriptdict are byte-identical before vs after, because every removed entry matched zero files. Verified with the real matcher (PyAutoHands/autobuild/build_util.py:143).Generated by the PyAutoLabs agent workflow.