Skip to content

feat: add dependency cooldown - #89

Merged
dewabisma merged 1 commit into
mainfrom
beast/add-dependency-cooldown
Aug 28, 2026
Merged

feat: add dependency cooldown#89
dewabisma merged 1 commit into
mainfrom
beast/add-dependency-cooldown

Conversation

@dewabisma

Copy link
Copy Markdown
Contributor

Basically adding the dependency cooldown workflow. Enforcing in repo rules will be done later because doesn't have permission.

@dewabisma
dewabisma requested a review from n13 August 28, 2026 06:03

jobs:
audit:
uses: Quantus-Network/shared-workflows/.github/workflows/dependency-cooldown-audit.yml@v1

jobs:
dependency-cooldown:
uses: Quantus-Network/shared-workflows/.github/workflows/dependency-cooldown.yml@v1

@n13 n13 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewer model: GPT Sol

APPROVE — the two callers match the shared v1 adoption templates, use least-privilege read permissions, and trigger on the events needed to re-evaluate emergency bypass metadata. No blocking findings.

Validation:

  • Reviewed the complete f885ad5...b79b94c diff and the referenced shared workflow/action at v1 (32aabd71).
  • The live dependency-cooldown / Dependency cooldown job passed on this head.
  • Ran the shared checker locally in PR mode: Cargo.lock found, 0 newly introduced versions, pass.
  • Ran audit mode over all 80 locked versions: pass, with the intended non-blocking warning for the existing 27-day-old ar_archive_writer@0.5.3.
  • Both YAML files parse cleanly; git diff --check passes; all current GitHub checks are green.

As noted in the PR description, the status check still needs to be made required in the repository ruleset before it becomes a merge gate; that administrative follow-up is outside this patch.

@dewabisma
dewabisma merged commit 0117e2c into main Aug 28, 2026
10 checks passed
@dewabisma
dewabisma deleted the beast/add-dependency-cooldown branch August 28, 2026 06:12
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