Shell script linter
@Benehiko is already working on this.
Since Jun 27, 2025.
Assessment
This issue has not been assessed yet.
Description
We should set up a shell script linter on the CI to reduce bugs:
- add a new CI job (it should follow the pattern, separate yml file ...)
- use shell check as linter
- make sure our existing shell files are linted (fixing the issues might require a follow up PR, depending how much there are)
We have something similar in the Docker CLI:
- Dominant language
- Go
- Stars
- 91
- Forks
- 16
- Avg merge
- 9h 3m
- Merged PRs (30d)
- 24
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from docker/secrets-engine
-
Linux keychain dials a new D-Bus connection per operation, defeating per-entry access confirmation Open
Difficulty 5/5 Over a week Newbie friendliness 42/100
docker/secrets-engine#648 · 1 comment ·
-
Difficulty 3/5 1-2 days Newbie friendliness 65/100
docker/secrets-engine#584 · 1 comment · 2 reactions ·
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
docker/secrets-engine#551 · 2 comments · 3 reactions ·
-
Difficulty 5/5 Over a week Newbie friendliness 30/100
docker/secrets-engine#534 · 16 comments · 4 reactions ·
-
Dynamic Scopes Open
docker/secrets-engine#393 · 1 assignee ·
All issues in docker/secrets-engine
Similar issues
-
bug github_actions
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
registrystack/registry-stack#1393 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
JakeChampion/lang#10213 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
oasisprotocol/oasis-sdk#2523 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100