Skip to content

fix(ci): remove excessive job-level permissions - #211

Closed
hyperpolymath wants to merge 1 commit into
mainfrom
fix/ci-timeout-minutes-reusable
Closed

fix(ci): remove excessive job-level permissions#211
hyperpolymath wants to merge 1 commit into
mainfrom
fix/ci-timeout-minutes-reusable

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Removes job-level permissions that exceed workflow-level permissions (Bug B).

The reusable workflow already declares its required permissions at the
workflow level, and job-level permissions in the caller that exceed
the workflow-level cap cause startup_failure.

Files fixed:

  • .github/workflows/secret-scanner.yml (removed pull-requests: write, actions: read)

Related: hyperpolymath/standards#426

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe vibe@mistral.ai

Removes job-level permissions that exceed workflow-level permissions.
The reusable workflow already declares its required permissions at the
workflow level, and job-level permissions in the caller that exceed
the workflow-level cap cause startup_failure.

Files fixed:
- .github/workflows/secret-scanner.yml (Bug B: removed pull-requests: write, actions: read)

Related: hyperpolymath/standards#426

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
@sonarqubecloud

Copy link
Copy Markdown

@gitar-bot

gitar-bot Bot commented Aug 12, 2026

Copy link
Copy Markdown

Important

You are using the Gitar free plan. Upgrade to unlock code review, CI analysis, auto-apply, custom automations, and more.

Gitar

@hyperpolymath
hyperpolymath enabled auto-merge (squash) August 12, 2026 20:32
@hyperpolymath
hyperpolymath disabled auto-merge August 12, 2026 21:03
@hyperpolymath
hyperpolymath enabled auto-merge (squash) August 12, 2026 21:03
auto-merge was automatically disabled August 12, 2026 22:05

Pull request was closed

@hyperpolymath
hyperpolymath deleted the fix/ci-timeout-minutes-reusable branch August 12, 2026 22:05
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