Skip to content

Move shellcheck out of changelog workflow into its own action#65

Closed
Copilot wants to merge 4 commits into
mainfrom
copilot/move-action-from-changelog
Closed

Move shellcheck out of changelog workflow into its own action#65
Copilot wants to merge 4 commits into
mainfrom
copilot/move-action-from-changelog

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 16, 2026

The shellcheck job was incorrectly placed in the changelog workflow despite having no relation to changelog validation.

Changes

  • .github/workflows/changelog.yaml — Removed shellcheck job; workflow now contains only check-changelog and check-changelog-commit
  • .github/workflows/shellcheck.yml (new) — Dedicated workflow for shell script linting, triggered on: push, with explicit permissions: contents: read

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@dannywillems dannywillems marked this pull request as ready for review March 16, 2026 18:51
…ck workflow

Co-authored-by: dannywillems <6018454+dannywillems@users.noreply.github.com>
Copilot AI changed the title [WIP] Move related action out of changelog Move shellcheck out of changelog workflow into its own action Mar 16, 2026
@dannywillems
Copy link
Copy Markdown
Contributor

@copilot add a changelog entry

Co-authored-by: dannywillems <6018454+dannywillems@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 16, 2026

@copilot add a changelog entry

Added a changelog entry in 57f7bca under [Unreleased] > Infrastructure.

@dannywillems
Copy link
Copy Markdown
Contributor

@copilot resolve conflicts

… workflow update

Co-authored-by: dannywillems <6018454+dannywillems@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 16, 2026

@copilot resolve conflicts

Resolved in b47f6d9. The branch was behind main by two commits (setup-uv v6→v7 bump in ci-per-commit.yaml and a matching CHANGELOG entry). Both are now merged into the branch alongside the shellcheck changes.

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.

2 participants