Standardize CI on the org shape - #4
Merged
Merged
Conversation
The checks move into .github/actions/test, called by CI. Nine of the fourteen gd-* repos carried a byte-identical ci.yml and five had drifted; the drift was almost entirely legitimate -- each addon validates its own file manifest -- so those manifests are preserved verbatim. The `if: hashFiles(...)` guards are gone. A step that skips itself when its test script is absent is indistinguishable from one whose script was renamed or deleted: a skipped test is a green tick. All fourteen repos were surveyed first and every one of them does run a suite, so removing the guards costs nothing today -- which is exactly why now is the cheapest time to remove them. Godot comes from a local setup-godot composite action, copied from the one castledrop and prizm already carry, rather than a download URL written into ci.yml. The engine version this addon is tested against was living in workflow YAML. One job named ci, timeout-minutes, and third-party actions pinned by SHA with the tag in a trailing comment. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Release also runs Test now. It used to read the version, package the addon and publish it to the GDAM registry without running any check at all -- the only thing between a broken commit and a published release was whether somebody had looked at CI first.
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.
Part of the
gd-*family standardization — fourteen repos, same change.What changes
.github/actions/test, called by CI. Nine of the fourteen carried a byte-identicalci.yml; five had drifted, almost entirely legitimately (each addon validates its own file manifest). Those manifests are preserved verbatim.if: hashFiles(...)guards are gone. A step that skips itself when its test script is absent is indistinguishable from one whose script was renamed or deleted — a skipped test is a green tick.setup-godotcomposite action, copied from the onecastledropandprizmcarry, rather than a download URL written intoci.yml.ci,timeout-minutes, third-party actions SHA-pinned.Why removing the guards is safe right now
All fourteen repos were surveyed first: every one of them does run a suite — twelve via
tests/test.sh, two via Go tests incli/. So removing the guards changes nothing today, which is exactly why now is the cheapest moment to do it. The next time someone renames a test script, it fails instead of going green.Phase 2 of the standardization plan (F4, F5, F6, Decisions 4–6).
Release runs Test now
It used to read the version, package the addon and publish it to the GDAM registry without running any check at all. The only thing between a broken commit and a published release was whether somebody had looked at CI first.