Skip to content

ci: run workflow lint on pull_request only#19

Merged
nbrieussel merged 1 commit into
mainfrom
ci/lint-workflows-pr-only
Jun 10, 2026
Merged

ci: run workflow lint on pull_request only#19
nbrieussel merged 1 commit into
mainfrom
ci/lint-workflows-pr-only

Conversation

@nbrieussel

Copy link
Copy Markdown
Contributor

Context

lint-workflows.yml triggered on both push and pull_request for .github/workflows/**. Since everything lands through a PR, the push run added no coverage — it only duplicated jobs and doubled exposure to transient failures (e.g. on #17 the push-triggered actionlint hit a GitHub API 500 while resolving the latest actionlint release, while the pull_request run passed).

Changes

  • Drop the push trigger from lint-workflows.yml; keep pull_request only. Mirrors the terraform-lock.yml workflow.

How to verify

  • actionlint .github/workflows/*.yml passes locally.
  • On this PR, actionlint + zizmor run once (no duplicate push run).

🤖 Generated with Claude Code

Everything lands through a PR, so the duplicate `push`-triggered run added
no coverage — it only doubled the jobs and the exposure to transient API
blips (e.g. a GitHub API 500 while raven-actions/actionlint resolves its
latest release). Mirror the terraform-lock workflow: pull_request only.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nbrieussel
nbrieussel marked this pull request as ready for review June 10, 2026 22:50
@nbrieussel
nbrieussel merged commit 51e8982 into main Jun 10, 2026
3 checks passed
@nbrieussel
nbrieussel deleted the ci/lint-workflows-pr-only branch June 10, 2026 22:50
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