chore: skip e2e tests for docs-only and ignorable-file PRs#7704
Conversation
Add IGNORABLE_PREFIXES and IGNORABLE_SUFFIXES to the e2e scope detection script so that PRs touching only documentation, changesets, AI config, or other non-rendering files skip e2e entirely instead of triggering the full 5-container Cypress suite. Changes: - Add SKIP sentinel return value to detectScope() for ignorable-only PRs - Add ignorable file detection (docs, changesets, .claude/, assistant/, etc.) - Update e2e.yml workflow to skip the e2e job when SKIP is returned - Update local runner (run-e2e-scoped.ts) to handle SKIP - Upgrade @argos-ci/cypress to ^6.3.3 for ARGOS_SUBSET support - Add 9 new test cases covering docs-only, mixed, and skip scenarios
|
✅ Deploy Preview for mermaid-js ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
@mermaid-js/examples
mermaid
@mermaid-js/layout-elk
@mermaid-js/layout-tidy-tree
@mermaid-js/mermaid-zenuml
@mermaid-js/parser
@mermaid-js/tiny
commit: |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #7704 +/- ##
==========================================
- Coverage 3.33% 3.33% -0.01%
==========================================
Files 542 542
Lines 56881 56928 +47
Branches 839 839
==========================================
Hits 1899 1899
- Misses 54982 55029 +47
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
Versions 6.3.x have a packaging bug where the exports map points to dist/task.js but the package ships dist/task.mjs instead, causing 'Cannot find module' errors in CI. Version 6.2.12 is the minimum version supporting ARGOS_SUBSET and ships the correct dist/task.js file.
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
Summary
Extends the e2e scope detection script (
scripts/e2e-diagram-scope.mjs) to recognize ignorable files — files that can never affect rendered diagram output. When a PR touches only ignorable files (docs, changesets, AI config, etc.), the entire e2e job is now skipped instead of running the full 5-container Cypress suite.This builds on the diagram-scoped e2e optimization from #7619.
Changes
scripts/e2e-diagram-scope.mjsIGNORABLE_PREFIXESlist covering: docs source/generated,.changeset/,.claude/,assistant/, docs CI workflows, doc scriptsIGNORABLE_SUFFIXES(.mdfiles at repo root) with a guard to exclude.mdfiles inside diagram source foldersSKIPsentinel return value (exported) when all changed files are ignorablecontinue(skipped) instead ofbreak(triggering full suite).github/workflows/e2e.ymldetect-scopestep now handles theSKIPoutput valuee2ejob has anif:condition to skip entirely whenSKIPis returnedscripts/run-e2e-scoped.tsSKIPwith early exit and clear messagescripts/e2e-diagram-scope.spec.tspackage.json@argos-ci/cypressto^6.2.12forARGOS_SUBSETsupport (minimum version with this feature). Versions 6.3.x have a packaging bug whereexportsmaps point todist/task.jsbut the package shipsdist/task.mjsinstead, causingCannot find moduleerrors in CI..cspell/code-terms.txtCODEOWNERSto the spell-check dictionaryDetection behavior matrix
SKIP""""Tests
All 25 tests pass (including 9 new ignorable-file tests).