Repository navigation
ci(hypatia): bump the pinned scanner to be98f74 in all three workflows - #262
Merged
Merged
Conversation
be98f74 is hypatia#970's merge commit. It brings in: - secret-scanning alert fetches that follow pagination and time out per request; - the local Scorecard licence check (SC-010) accepting LICENSE.adoc and REUSE LICENSES/, in agreement with root hygiene; - REUSE LICENSES/ files that count only when named by an SPDX identifier or LicenseRef-; - RE001 scoped to credentials exposed to code the job cannot vouch for (hypatia#966). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01P4yFkxW5Heb8p2YW7ZYs9F
Contributor
|
Caution Review failedThe pull request is closed. Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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.



Summary
This moves the template's pinned Hypatia commit from
7c10e5d(#261) tobe98f74, the merge commit of hyperpolymath/hypatia#970. It brings in four scanner changes:LICENSE.adocand REUSELICENSES/, so it agrees with root hygiene. Before, one scan could report both "licence present" and "No LICENSE file found".LICENSES/: a file counts only if its name is an SPDX identifier (vendored list, SPDX 3.29.0) or aLicenseRef-id. A strayLICENSES/notes.txtno longer passes.Changes
.github/workflows/hypatia-scan.yml:HYPATIA_SHA. The cache key follows it..github/workflows/static-analysis-gate.yml: the escript cache key and the fetch..github/workflows/dogfood-gate.yml: the checkoutrefand the oracle cache key.7c10e5dappeared nowhere else in the repository.RSR Quality Checklist
Required
Hypatia Security Scan,static-analysis-gateanddogfood-gateruns exercise the new pin. Those are the checks that matter here.unsafeblocks without// SAFETY:comments.envfiles includedAs Applicable
None apply: this is a pin bump with no change to state, integrations, architecture or dependencies.
Testing
7c10e5doccurrence is replaced (grepfinds none left), and the three workflows parse.Downstream: repositories synced from the template, including echo-types since hyperpolymath/echo-types#356, keep
7c10e5duntil they re-sync.Generated by Claude Code