← Back to issue list

CI to check pre-commit updates from renovate

View original Github issue

Metadata

Project
starbase
Number
#352
Type
issue
State
open
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

CI should run pre-commit run --all-files when renovate updates .pre-commit-config.yaml. Issue is 565 days old but had a Copilot research comment 19 days ago confirming the gap still exists. A prior attempt to add --all-files was walked back (commit 2443e63), so the solution needs rethinking.

Suggested action: keep open

Reason: The Copilot research comment from 19 days ago explicitly confirms no CI job runs pre-commit run --all-files and that make lint bypasses the pre-commit framework entirely. The walk-back commit 2443e63 (fix: walk back pre-commit all-files) shows the naive approach was rejected, but the underlying gap remains. The issue is a legitimate CI improvement request that needs a maintainer to decide on the right approach.

Impact: 35 Quick Win: 21.0 Staleness: 45 Complexity: 40 Confidence: 65 Support Request: 10

Issue body

When renovate updates the pre-commit config, we should run `pre-commit run --all-files` in CI to ensure it doesn't break things.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.8-27b
Impact: 35
Quick Win: 21.0
Staleness: 45
Complexity: 40
Confidence: 65
Support Request: 10
keep open CI should run pre-commit run --all-files when renovate updates .pre-commit-config.yaml. Issue is 565 days old but had a Copilot research comment 19 days ago confirming the gap still exists. A prior attempt to add --all-files was walked back (commit 2443e63), so the solution needs rethinking.
qwen/qwen3.6-35b-a3b
Staleness: 85
Complexity: 20
Confidence: 85
Support Request: 10
needs triage Request to add a CI job running pre-commit on Renovate updates, as current CI bypasses the framework. Awaiting maintainer triage.
qwen/qwen3.6-35b-a3b
Staleness: 95
Complexity: 15
Confidence: 85
Support Request: 5
needs triage Request to add a CI job running pre-commit checks after Renovate updates. Unlabelled, unassessed, and inactive for over 14 months with zero maintainer interaction.
qwen3.6-35b-a3b-mtp-q6
Staleness: 90
Complexity: 15
Confidence: 80
Support Request: 5
needs triage Request to add a CI step that runs pre-commit checks when Renovate updates the pre-commit configuration. The issue remains unlabelled and unaddressed after over a year.
qwen3.6-35b-a3b-mtp-q6
Staleness: 90
Complexity: 10
Confidence: 80
Support Request: 0
close stale Implement a CI job to run pre-commit checks after Renovate updates the configuration, ensuring changes do not break the build. Open, unassigned, and inactive for 489 days.

Update history

Date Change
updated

Related work

  • Related To: starbase#242 (confidence 80%)

    Merged PR that enabled Renovate pre-commit manager support, which is the prerequisite for this issue's CI check to be meaningful.

  • Related To: starbase#185 (confidence 70%)

    Merged fix for the pre-commit dependency manager in CI/renovate config, part of the same renovate+pre-commit integration effort.

Related issues

No related issues found above the similarity threshold.