chore: add static checks workflow for #320 phase C#325
Conversation
|
Phase C PR is ready for review and labels are synced with issue #320.
|
There was a problem hiding this comment.
Pull request overview
Adds CI static analysis for shell scripts and GitHub Actions workflows as part of the Phase C transition plan in #320, while bumping versioning/docs to reflect the new milestone.
Changes:
- Added a new CI workflow to run
shellcheck,shfmt -d, andactionlinton relevant changes. - Updated the bsctl/CodeQL decommission planning doc with Phase C status and next steps.
- Bumped version to
0.1.23and added a correspondingCHANGELOG.mdentry (including legacy mirror update).
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/static-checks.yaml |
Introduces CI jobs for shell/workflow static checks to replace Go-focused scanning over time. |
docs/plans/bsctl-codeql-decommission-plan.md |
Records Phase C status and refines immediate next steps around static-check stabilization. |
resources/version.yaml |
Updates the primary repo version source to 0.1.23. |
bsctl/static/resources/constants.yaml |
Mirrors the version bump to 0.1.23 for legacy transition compatibility. |
CHANGELOG.md |
Adds a 0.1.23 entry documenting the workflow/doc updates. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Andrew Shoell <mrlunchbox777@gmail.com>
69d41a9 to
5ef8d8e
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Andrew Shoell <mrlunchbox777@gmail.com>
Signed-off-by: Andrew Shoell <mrlunchbox777@gmail.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Andrew Shoell <mrlunchbox777@gmail.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Andrew Shoell <mrlunchbox777@gmail.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Andrew Shoell <mrlunchbox777@gmail.com>
Summary
.github/workflows/static-checks.yamlto runshellcheck,shfmt -d, andactionlintin CI0.1.26inresources/version.yaml, mirroredbsctl/static/resources/constants.yaml, and newCHANGELOG.mdentryValidation
./scripts/workflows/docs-bump_docs-bump_version-bump.sh./scripts/workflows/docs-bump_docs-bump_CHANGELOG-bump.shIssue Link