Skip to content

ci: optimize test workflow time#863

Merged
fi3ework merged 1 commit intomainfrom
ci-time
Jan 15, 2026
Merged

ci: optimize test workflow time#863
fi3ework merged 1 commit intomainfrom
ci-time

Conversation

@fi3ework
Copy link
Copy Markdown
Member

@fi3ework fi3ework commented Jan 13, 2026

Summary

  • introduce prepare job to gate ut/e2e via paths-filter
  • switch pnpm setup to setup-node with fetch/offline install
  • cache Playwright browsers per OS using PLAYWRIGHT_BROWSERS_PATH
  • reduce PR e2e matrix to node 18/24 while cron keeps full matrix (add daily scheduled full matrix at 00:00 CST to run full node version matrix)

before

image

now (without cache)

image

now (with cache)

image

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@netlify
Copy link
Copy Markdown

netlify bot commented Jan 13, 2026

Deploy Preview for rstest-dev ready!

Name Link
🔨 Latest commit 4f79bcf
🔍 Latest deploy log https://app.netlify.com/projects/rstest-dev/deploys/69663d4bcdf73600083ecc64
😎 Deploy Preview https://deploy-preview-863--rstest-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@fi3ework fi3ework force-pushed the ci-time branch 4 times, most recently from 834e07d to 3368d8e Compare January 13, 2026 11:45
@fi3ework fi3ework marked this pull request as ready for review January 13, 2026 12:04
Copilot AI review requested due to automatic review settings January 13, 2026 12:04
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 optimizes the CI test workflow to reduce execution time for pull requests while maintaining comprehensive testing on scheduled runs. It introduces a prepare job with path filtering to skip tests for documentation-only changes, modernizes the pnpm setup approach, adds Playwright browser caching, and reduces the Node.js version matrix for PR tests.

Changes:

  • Added a prepare job with paths-filter to gate test execution based on file changes
  • Modernized pnpm setup by switching from manual corepack installation to pnpm/action-setup with fetch/offline install
  • Implemented Playwright browser caching per OS to avoid repeated browser downloads
  • Reduced e2e test matrix for PRs to Node.js 18/24 only, while scheduled runs test all versions (18/20/22/24)
  • Added daily scheduled workflow run at 16:00 UTC (00:00 CST)

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

@fi3ework fi3ework merged commit b2f7ba3 into main Jan 15, 2026
12 checks passed
@fi3ework fi3ework deleted the ci-time branch January 15, 2026 12:22
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.

3 participants