Skip to content

ci: guard scorecard behind SCORECARD_ENABLED repo var - #628

Open
arcaven wants to merge 1 commit into
developfrom
ci/scorecard-guard
Open

ci: guard scorecard behind SCORECARD_ENABLED repo var#628
arcaven wants to merge 1 commit into
developfrom
ci/scorecard-guard

Conversation

@arcaven

@arcaven arcaven commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

Makes Scorecard analysis opt-in: the job runs only when the repo variable SCORECARD_ENABLED is set to true — the same opt-in pattern already used for SIGNING_ENABLED. No var, no run. Forks and clones stay quiet by default; enable per repo by adding the variable.

@arcaven
arcaven requested a review from Zious11 as a code owner July 17, 2026 17:00
Zious11 added a commit to Zious11/wirerust that referenced this pull request Jul 19, 2026
…414)

Adds the OSSF Scorecard workflow (same shape as jira-cli's
`scorecards.yml`, see Zious11/jira-cli#628) with the analysis job opt-in
from day one: it runs only when the repo variable `SCORECARD_ENABLED` is
set to `true` — the same opt-in pattern already used for
`SIGNING_ENABLED`. No var, no run. Forks and clones stay quiet by
default; enable per repo by adding the variable.

Co-authored-by: Jared Richards <jaredbrichards@gmail.com>
@arcavenai

Copy link
Copy Markdown

Diagnosed the two failing checks — unrelated to this PR's change (the diff is 4 lines in scorecards.yml; the failure is in Spec Guards' BC citation check):

Fix: update the branch against develop (the GitHub "Update branch" button suffices — the branch touches only scorecards.yml, so it's conflict-free) and let checks re-run. This account doesn't have write access to push the prepared merge, so flagging for someone who does.

@arcavenai

Copy link
Copy Markdown

To make the fix copy-paste executable (this account can't push to this branch): no file content changes are needed — the remedy is purely a branch refresh, since the DEAD citations name files that exist on current develop (e33624cc28ae94, the four S-576 attachment PRs) but not in this branch's stale merge ref.

Either click Update branch on this PR, or:

git fetch origin develop ci/scorecard-guard
git checkout ci/scorecard-guard
git merge origin/develop        # conflict-free — this branch touches only .github/workflows/scorecards.yml
git push origin ci/scorecard-guard

Verified locally on exactly that merge: mounted factory-artifacts at .factory/ and ran bash scripts/check-bc-citation-symbols.shCheck passed: 354 citations checked (exit 0), and the PR's own diff remains the 4-line SCORECARD_ENABLED gate.

@arcaven
arcaven requested a review from Zious11 July 23, 2026 00:57
Scorecard analysis is now opt-in: the job runs only when the repo
variable SCORECARD_ENABLED is set to true (same opt-in pattern as
SIGNING_ENABLED). No var, no run — forks and clones stay quiet by
default.
@arcaven
arcaven force-pushed the ci/scorecard-guard branch from 7584b52 to 2d5ae74 Compare August 4, 2026 22:57
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.

3 participants