Skip to content

Use generic workflow discovery in lint-workflows - #1776

Merged
v1v merged 1 commit into
mainfrom
fix-1775-actionlint-discovery-698287b990673dbb
Aug 27, 2026
Merged

Use generic workflow discovery in lint-workflows#1776
v1v merged 1 commit into
mainfrom
fix-1775-actionlint-discovery-698287b990673dbb

Conversation

@github-actions

@github-actions github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace the hand-maintained .github/workflows filename allowlist in make lint-workflows with generic workflow discovery for all top-level workflow YAML files
  • keep generated workflow outputs excluded by ignoring *.lock.yml / *.lock.yaml
  • preserve linting for claude-workflows/**/example.yml files
  • temporarily exclude .github/workflows/agentics-maintenance.yml because it currently contains a pre-existing actionlint syntax error (string should not be empty)

Validation

  • make lint-workflows

Resolves #1775.


What is this? | From workflow: Trigger Framework Best Practices

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@pushkarkurkure172-crypto

Copy link
Copy Markdown

This PR addresses #1775 by migrating the workflow linting process from a manual allowlist to a dynamic, generic discovery model.

Key Updates:

  • Replaced the hand-maintained list in make lint-workflows with global workflow discovery.
  • Excluded generated output lockfiles (*.lock.yml / *.lock.yaml).
  • Maintained coverage for claude-workflows/**/example.yml.
  • Temporarily bypassed .github/workflows/agentics-maintenance.yml to prevent failures from a pre-existing empty string syntax error.

Current Status:

  • Draft PR awaiting maintainer review and workflow run approval.

@v1v
v1v marked this pull request as ready for review August 27, 2026 09:24
v1v

This comment was marked as resolved.

@github-actions github-actions Bot added the medium_boom Medium PR blast radius; likely benefits from human review label Aug 27, 2026
@v1v
v1v merged commit 824766d into main Aug 27, 2026
19 of 23 checks passed
@v1v
v1v deleted the fix-1775-actionlint-discovery-698287b990673dbb branch August 27, 2026 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

medium_boom Medium PR blast radius; likely benefits from human review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[framework-best-practices] Use actionlint native workflow discovery instead of hand-maintained workflow allowlist

2 participants