Skip to content

fix(ci): drop illegal timeout-minutes on uses: jobs (mirror, secret-scanner) - #58

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/uses-job-timeout
Aug 3, 2026
Merged

fix(ci): drop illegal timeout-minutes on uses: jobs (mirror, secret-scanner)#58
hyperpolymath merged 1 commit into
mainfrom
fix/uses-job-timeout

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Same defect class as #56, which missed these two files. NOTE: repo-wide startup_failure (GitHub lockfile enforcement) means no checks will run on this PR; verified by file inspection. Merge after the owner's gh-actions-lock fix, or on inspection.

A job whose body is 'uses:' may not carry timeout-minutes; the key
invalidates the entire file (conclusion 'failure' at 0s, run named by
file path). PR #56 fixed scorecard.yml and missed these two.
@gitar-bot

gitar-bot Bot commented Aug 3, 2026

Copy link
Copy Markdown

Note

Automatic reviews are paused because your trial's included automatic processing has been used for this period. Upgrade now, or comment "Gitar review" to run a review anytime.
Learn more

Code Review ✅ Approved

Removes illegal timeout-minutes settings from reusable-workflow jobs in the CI configuration. No issues found.

Auto-approved and auto-merge armed: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria. — merges when pipeline and required approvals pass.

Options

Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Compact
gitar display:verbose         

Important

Your trial ends in 7 days — upgrade now to keep code review, CI analysis, auto-apply, custom automations, and more.

Was this helpful? React with 👍 / 👎 | Gitar

@gitar-bot
gitar-bot Bot enabled auto-merge (squash) August 3, 2026 16:37

@gitar-bot gitar-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Gitar has auto-approved this PR and enabled auto-merge (configure)

@gitar-bot gitar-bot Bot added the gitar-approved Added by Gitar label Aug 3, 2026
@hyperpolymath
hyperpolymath disabled auto-merge August 3, 2026 16:42
@hyperpolymath
hyperpolymath merged commit 46ea226 into main Aug 3, 2026
1 check passed
@hyperpolymath
hyperpolymath deleted the fix/uses-job-timeout branch August 3, 2026 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gitar-approved Added by Gitar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant