Skip to content

ci: Add GitHub Actions workflow linting with actionlint#2

Merged
nbrieussel merged 4 commits intomainfrom
ci/add-actionlint-workflow
Mar 22, 2026
Merged

ci: Add GitHub Actions workflow linting with actionlint#2
nbrieussel merged 4 commits intomainfrom
ci/add-actionlint-workflow

Conversation

@nbrieussel
Copy link
Copy Markdown
Contributor

Contexte

This PR adds automated linting of GitHub Actions workflows to ensure syntax correctness and best practices compliance.

Changements

  • Dependency: Added actionlint 1.7 to mise.toml for local workflow validation
  • Workflow: Created .github/workflows/lint-workflows.yml
    • Triggers on push and PR to any workflow files
    • Uses actionlint to lint all YAML workflow files in .github/workflows/
    • Fails if any linting errors are found

Bénéfices

  • Early detection of workflow syntax errors
  • Enforces GitHub Actions best practices
  • Prevents broken workflows from being merged

Tests

  • Run locally: actionlint .github/workflows/*.yml
  • Check that lint workflow appears in GitHub Actions on PR

- Add copilot-instructions.md with project guidelines
- Add CONTRIBUTING.md with branch, commit, and PR conventions
- Add hooks for automatic mise install on mise.toml changes
- Update README.md with onboarding instructions
- Add terraform 1.9 to mise.toml dependencies
- Add actionlint 1.7 to mise.toml
- Create ci workflow to lint all GitHub Actions workflows
- Runs on push and PR to workflow files
- Ensures workflow syntax and best practices compliance
@nbrieussel nbrieussel force-pushed the ci/add-actionlint-workflow branch from f5f1378 to bb2fa80 Compare March 22, 2026 13:39
@nbrieussel nbrieussel marked this pull request as ready for review March 22, 2026 13:42
@nbrieussel nbrieussel merged commit c68680c into main Mar 22, 2026
2 checks passed
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