oxidecomputer / oxidecomputer/dendrite
Include `tools` dir in shellcheck CI
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 20
- Forks
- 3
- Avg merge
- 3d 11h
- Merged PRs (30d)
- 4
Description
I'm setting up shellcheck for buildomat scripts in https://github.com/oxidecomputer/dendrite/pull/367
There are also scripts in the tools dir that should probably be linted. However, migrating those implies meaningful validation scope and should probably be a standalone project.
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.
Research direction
Review the shellcheck CI setup for the buildomat scripts referenced in PR 367, then inventory the scripts under the tools directory. Determine the validation scope needed for those scripts and confirm how the CI should include them. Done means the tools scripts are covered by shellcheck CI with the intended scope validated.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github-actions, shell
- Domain
- ci-cd, tooling
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 58/100