Skip to content

fix(ci): ignore actionlint false positive for code-quality scope#244

Merged
devantler merged 1 commit into
mainfrom
claude/actionlint-code-quality-ignore
May 26, 2026
Merged

fix(ci): ignore actionlint false positive for code-quality scope#244
devantler merged 1 commit into
mainfrom
claude/actionlint-code-quality-ignore

Conversation

@devantler
Copy link
Copy Markdown
Contributor

What

Adds ACTION_ACTIONLINT_ARGUMENTS: -ignore code-quality to the validate-go-project MegaLinter step.

Why

MegaLinter bundles actionlint v1.7.12, which doesn't yet know the code-quality permission scope (added for GitHub Code Quality coverage). Since validate-go-project lints consumer workflows, any Go repo that grants code-quality: write in its own workflow (e.g. ksail's ci-go caller) fails MegaLinter with a false unknown permission scope "code-quality" error. This ignores that one false positive until actionlint catches up. Discovered on ksail#4893.

zizmor finding-neutral; yamllint clean.

🤖 Generated with Claude Code

MegaLinter's bundled actionlint (<=1.7.12) doesn't yet recognize the
`code-quality` permission scope that GitHub Code Quality coverage requires
callers to grant, so it fails consumer workflows that use it (e.g. ksail's
ci-go). Pass `-ignore code-quality` to actionlint until it catches up.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 26, 2026 16:45
@devantler devantler enabled auto-merge (squash) May 26, 2026 16:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the validate-go-project reusable workflow to avoid a known actionlint false positive when consumer repositories use the newer code-quality permission scope (used for GitHub Code Quality coverage), which MegaLinter’s bundled actionlint version does not yet recognize.

Changes:

  • Adds ACTION_ACTIONLINT_ARGUMENTS: -ignore code-quality to MegaLinter’s environment in validate-go-project.
  • Documents the rationale inline so the workaround is easy to remove once MegaLinter/actionlint is updated.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@devantler devantler merged commit e173ff6 into main May 26, 2026
37 checks passed
@devantler devantler deleted the claude/actionlint-code-quality-ignore branch May 26, 2026 16:49
@github-project-automation github-project-automation Bot moved this from 🫴 Ready to ✅ Done in 🌊 Project Board May 26, 2026
@botantler
Copy link
Copy Markdown
Contributor

botantler Bot commented May 26, 2026

🎉 This PR is included in version 4.0.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@botantler botantler Bot added the released an issue that has been solved in a release label May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released an issue that has been solved in a release

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants