Skip to content

ci: upload download link audit results as an artifact - #1998

Merged
dsanders11 merged 3 commits into
mainfrom
audit-results-artifact
Aug 7, 2026
Merged

ci: upload download link audit results as an artifact#1998
dsanders11 merged 3 commits into
mainfrom
audit-results-artifact

Conversation

@claude

@claude claude Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Requested by Samuel Attard · Slack thread

Before: the results of the nightly download-link check (the broken-links table or the "no broken links" note) only exist on the workflow run's step summary page, which is not retrievable through any GitHub API — so automated tooling and agents cannot read the findings.

After: the same findings markdown is also uploaded as a build artifact named audit-results, downloadable via the GitHub API. The upload uses archive: false, so the artifact is a single raw file listed under the file name audit-results.md, and the REST artifact endpoint serves the raw markdown directly — no zip extraction needed. The step summary, exit code, and all other behavior are unchanged.

How: immediately before await core.summary.write() (which empties the summary buffer), the script writes the buffered markdown to audit-results.md via core.summary.stringify(). A new actions/upload-artifact step (pinned to the same SHA the repo already uses in release.yml, v7.0.1) uploads that file with archive: false and if: always() so the artifact is produced whether or not broken links were found and the step failed.


Generated by Claude Code

@MarshallOfSound
MarshallOfSound marked this pull request as ready for review August 6, 2026 21:55
@MarshallOfSound
MarshallOfSound requested review from a team and codebytere as code owners August 6, 2026 21:55
@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 87.352%. remained the same — audit-results-artifact into main

@dsanders11
dsanders11 merged commit 1e36e19 into main Aug 7, 2026
17 checks passed
@dsanders11
dsanders11 deleted the audit-results-artifact branch August 7, 2026 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants