Skip to content

ci: lint scripts and workflows#48

Open
janisz wants to merge 3 commits intomainfrom
shellcheck
Open

ci: lint scripts and workflows#48
janisz wants to merge 3 commits intomainfrom
shellcheck

Conversation

@janisz
Copy link
Contributor

@janisz janisz commented Mar 2, 2026

added shellcheck and actionlint

janisz and others added 2 commits March 2, 2026 14:25
- Fix unused loop variable warnings in shell scripts by using underscore
- Fix codecov action parameter (file -> files) for v5 compatibility

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Add comprehensive linting for shell scripts and GitHub Actions workflows to improve code quality and catch issues early.

- Add shellcheck configuration (.shellcheckrc) with external sources enabled
- Add actionlint for GitHub Actions workflow validation
- Create make targets for shell-lint and actionlint
- Integrate both linters into CI style workflow
- Add documentation for code style checks and shell script guidelines
- Add actionlint as a versioned Go tool dependency

The project now has linting coverage for:
- Go code (golangci-lint)
- Shell scripts (shellcheck)
- GitHub Actions workflows (actionlint)
- Dockerfiles (hadolint)
- Helm charts (kube-linter)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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