Repository navigation
fix(audit): read the catalogue registry from site/prepared and admit docs/evidence images - #1440
Merged
Merged
Conversation
…docs/evidence images
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.
Change
The "Check documentation links and organization" step of the repository audit fails on main. Two refactor moves left it behind:
site/directory/objects.mtsnow imports the catalogue from../prepared/prepared-catalogue.mjs(refactor(site): write generated site outputs into site/prepared #1358). The provenance inventory only recognised./prepared-…, found no registered bodies and stopped. It now accepts any relative path to the prepared catalogue and reads 5,317 registered bodies on main.docs/evidence/(refactor(docs): move the evidence images into docs/evidence/ #1409). The docs checker admitted onlydocs/images/and refused all eleven files. It now admits images underdocs/evidence/and each capture'sreceipt.json; other files there are still refused.Evidence and limits
Proves the change: each new test fails against the old script and passes with the fix:
test_catalogue_import_from_the_prepared_folder_is_recognisedandevidence images and their capture receipt are admitted under docs/evidence/, other files are not.Limits: the step stays red on one remaining finding owned by the stellar corona work:
docs/stellar-corona-from-magnetic-maps.mdlinkspackages/bake/src/objects/stellar/corona/models.test.mts, which does not exist. The "directory grew past its baseline" step is a separate failure and is not changed here.