Skip to content

ci: strengthen workflows, bump actions to v1.7.1 and add dependabot web coverage - #24

Merged
kleinbem merged 1 commit into
mainfrom
ci-strengthen
Nov 10, 2026
Merged

kleinbem merged 1 commit into
mainfrom
ci-strengthen

Conversation

@kleinbem

@kleinbem kleinbem commented Nov 10, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Add /web npm coverage to .github/dependabot.yml.
  • Bump nix-setup, fleet-lint, and maintain-flake to v1.7.1 (f1b4f5fc52ea31b1f9e53ca29170570fe4651c7d).
  • Add workflow concurrency to avoid redundant runs on superseded commits.

Summary by CodeRabbit

  • Chores
    • Added scheduled checks for web dependencies, with minor and patch updates grouped together.
    • Updated maintenance and validation workflows, and set newer runs to cancel older in-progress runs for the same workflow and branch.
    • No changes to user-facing functionality.

@kleinbem
kleinbem enabled auto-merge (squash) November 10, 2026 14:30
@coderabbitai

coderabbitai Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The pull request adds weekly npm dependency update settings and updates GitHub Actions workflow concurrency and pinned revisions.

Changes: Dependency update configuration

Layer / File(s) Summary
npm update settings
.github/dependabot.yml
Adds weekly Dependabot updates for npm packages in /web, a seven-day default cooldown, and a group for minor and patch updates.

Workflow concurrency and pins

Layer / File(s) Summary
Check workflow concurrency
.github/workflows/check.yaml
Groups runs by workflow and ref. A new run cancels an in-progress run in the same group.
Workflow revision updates
.github/workflows/check.yaml, .github/workflows/fleet-lint.yaml, .github/workflows/maintain.yaml
Updates the Nix setup pins in the check and fleet-lint workflows, and the reusable workflow pin in the maintain workflow, to v1.7.1.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Possibly related PRs

  • kleinbem-dev/cadre#11: Adds the fleet-lint workflow and pins its lint and Nix setup actions, which this pull request updates.
  • kleinbem-dev/cadre#12: Updates the same fleet-lint action pins that this pull request advances to v1.7.1.
  • kleinbem-dev/cadre#19: Adds the maintain workflow caller whose reusable workflow pin this pull request updates.
Merge Risk: Minimal · up to bf434

No actionable merge-blocking risk was identified in the automation changes.

Architecture risk: Low · up to bf434

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

  • observed — Modified behavior in .github/dependabot.yml: Adds weekly Dependabot updates for npm packages in /web, applies a seven-day default cooldown, and groups all packages’ minor and patch updates under npm-dependencies.
  • observed — Modified behavior in .github/workflows/check.yaml: Adds concurrency control grouped by workflow and ref; a new run cancels an in-progress run in the same group.
  • observed — Modified behavior in .github/workflows/check.yaml: Updates the pinned kleinbem/actions/nix-setup revision from v1.0.1 to v1.7.1 in the check job.
  • observed — Modified behavior in .github/workflows/check.yaml: Updates the pinned kleinbem/actions/nix-setup revision from v1.0.1 to v1.7.1 in the push-cache job.
Pre-merge checks: Passed 5

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly identifies the workflow updates, action version bumps, and Dependabot coverage for /web.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.)

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Central YAML; Path: .coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 11a9d577-e1ba-47bd-9277-4cd39c1b4ddc
Commits

Reviewing files that changed from the base of the PR and between 936f99c and bf43434.

Files selected for processing: 4

  • .github/dependabot.yml
  • .github/workflows/check.yaml
  • .github/workflows/fleet-lint.yaml
  • .github/workflows/maintain.yaml

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 7 remain after this review.

@kleinbem
kleinbem merged commit a31e772 into main Nov 10, 2026
4 checks passed
@kleinbem
kleinbem deleted the ci-strengthen branch November 10, 2026 15:36
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.

1 participant