Skip to content

ci: split into multiple files#12

Merged
AsakuraMizu merged 3 commits intomainfrom
ci
Feb 6, 2026
Merged

ci: split into multiple files#12
AsakuraMizu merged 3 commits intomainfrom
ci

Conversation

@AsakuraMizu
Copy link
Contributor

Copy link

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 refactors the CI/CD pipeline by splitting a monolithic .github/workflows/ci.yml file into two separate workflow files: check.yml for code quality checks and testing, and docs.yml for documentation building and deployment. This improves maintainability and allows the workflows to run independently with different triggers and configurations.

Changes:

  • Split CI workflow into separate check and documentation workflows
  • Updated GitHub Actions versions from v4 to v6 for checkout action
  • Changed documentation deployment from JamesIves/github-pages-deploy-action to official GitHub Pages actions
  • Modified workflow triggers: docs now only runs on main branch pushes, while checks run on all pushes and PRs

Reviewed changes

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

File Description
.github/workflows/ci.yml Removed monolithic CI workflow file
.github/workflows/check.yml New workflow for code formatting, linting (clippy), and testing across multiple Rust toolchain versions
.github/workflows/docs.yml New workflow for building and deploying documentation to GitHub Pages

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

Copy link

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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.


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

@AsakuraMizu AsakuraMizu merged commit d59fa3b into main Feb 6, 2026
14 checks passed
@AsakuraMizu AsakuraMizu deleted the ci branch February 6, 2026 07:28
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.

1 participant