Skip to content

ci: update workflows#484

Merged
jannik-stacks merged 6 commits intomainfrom
ci/update-pr-flow
Dec 30, 2025
Merged

ci: update workflows#484
jannik-stacks merged 6 commits intomainfrom
ci/update-pr-flow

Conversation

@jannik-stacks
Copy link
Copy Markdown
Collaborator

  • Tighten workflows and actions
  • Comment broken version step
  • Reuse actions/setup
  • Pin third-party workflows

Copy link
Copy Markdown

@albu-stacks albu-stacks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! The action runs look fine too

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR modernizes and streamlines the GitHub Actions workflows by consolidating common functionality into reusable actions, pinning third-party actions to specific commit hashes for security, and temporarily disabling a broken version publishing step.

Key changes:

  • Refactored setup and code check steps into reusable composite actions (install-node and check-code)
  • Pinned third-party action wagoid/commitlint-github-action to a specific commit hash for security
  • Temporarily commented out the automatic version publishing job due to permissions issues

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
.github/workflows/version.yml Renamed workflow, added concurrency control, refactored to use new composite actions, commented out broken version job
.github/workflows/pull_request.yml Deleted old pull request workflow file
.github/workflows/pull-request.yml New simplified pull request workflow using reusable actions and workflows, with pinned third-party actions
.github/workflows/publish.yml Renamed workflow, added concurrency control, refactored to use new install-node action, removed unnecessary pull-requests permission
.github/workflows/pre-run.yml New reusable workflow for fork detection that can be called by other workflows
.github/actions/install-node/action.yml New composite action that sets up Node.js 24 and installs dependencies
.github/actions/code-checks/action.yml Deleted old code checks action
.github/actions/check-code/action.yml New composite action for running linting and type checking (renamed from code-checks)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jannik-stacks jannik-stacks merged commit f9c3c7d into main Dec 30, 2025
12 checks passed
@jannik-stacks jannik-stacks deleted the ci/update-pr-flow branch December 30, 2025 12:04
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.

4 participants