Skip to content

Conversation

@cusma
Copy link
Collaborator

@cusma cusma commented Nov 14, 2025

This PR improves docs CI/CD:

  1. Shards SC tests scoped by directory (addresses feat: parallelize tests #223);
  2. Deploys docs on release (addresses fix: disable docs deployment on main-release sync #286);
  3. Deploys docs preview on docs-related PRs;
  4. Updates mdbook version.

@cusma cusma self-assigned this Nov 14, 2025
@cusma cusma added the ci-cd label Nov 14, 2025
Copilot finished reviewing on behalf of cusma November 14, 2025 15:02
Copy link
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 enhances the documentation CI/CD pipeline by adding automated deployment on releases and PR preview functionality, while also updating the mdbook version to 0.4.52.

Key Changes:

  • Implements docs deployment triggered by successful mainnet releases via workflow_run
  • Adds PR preview deployment for docs-related changes with path filtering
  • Updates mdbook from version 0.4.43 to 0.4.52 across all workflows

Reviewed Changes

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

File Description
.github/workflows/docs-ci.yaml Adds GitHub Pages preview deployment for PRs, includes path filters for docs-related files, and configures concurrency management
.github/workflows/docs-cd.yaml Changes trigger from push to workflow_run based on release workflow completion, updates mdbook version, and adds conditional execution
.github/actions/docs-ci/action.yaml Updates markdown file glob patterns to include SECURITY.md and use recursive pattern, updates mdbook version

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

@cusma cusma changed the title ci-cd: docs deployment preview and release ci-cd: parallel tests, docs deployment preview and release Nov 14, 2025
@cusma cusma requested a review from Copilot November 17, 2025 18:42
Copilot finished reviewing on behalf of cusma November 17, 2025 18:46
Copy link
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

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


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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: disable docs deployment on main-release sync feat: parallelize tests

2 participants