Skip to content

ci: deploy-specific build path and doc-only path filters#5009

Open
PascalThuet wants to merge 1 commit into
Dash-Industry-Forum:developmentfrom
PascalThuet:feature/ci-cd-p3-build-only-and-path-filters
Open

ci: deploy-specific build path and doc-only path filters#5009
PascalThuet wants to merge 1 commit into
Dash-Industry-Forum:developmentfrom
PascalThuet:feature/ci-cd-p3-build-only-and-path-filters

Conversation

@PascalThuet
Copy link
Copy Markdown
Contributor

@PascalThuet PascalThuet commented Apr 3, 2026

Summary

Implements the remaining reusable-workflow and CI-trigger parts of P3 from #5004.

  • adds a deploy-specific build:dist script: rimraf dist && tsc && webpack, without tests or lint
  • switches reusable deploy.yml to build:dist for nightly and v4 deploys
  • adds path filters so verify_pull_request and verify_branch skip when changes are limited to:
    • root *.md
    • docs/**
    • proposals/**
    • LICENSES/**
  • together with ci: consolidate master deploys into a single workflow (4 builds → 1) #5008, this completes the planned P3 changes

Note: if verify_pull_request is a required check for merging, paths-ignore may leave doc-only PRs in a pending state. In that case, a lightweight passthrough job or a paths-based strategy may still be needed.

Test plan

  • npm run build:dist produces the expected deploy artifacts without running test/lint
  • Nightly and v4 deploy workflows complete successfully with build:dist
  • A PR touching only root markdown, docs/, proposals/, or LICENSES/ does not trigger verification
  • A PR touching code plus docs still triggers CI normally
  • Required-check behavior is acceptable for doc-only PRs

Refs #5004

🤖 Generated with Claude Code

- Add build:dist script (tsc + webpack, skips tests and lint) for use
  in deploy workflows where code is already verified.
- Update deploy.yml to use build:dist instead of build.
- Add paths-ignore to verify_pull_request and verify_branch so that
  changes limited to markdown files, docs/, proposals/, or LICENSES/
  do not trigger CI builds.

Refs Dash-Industry-Forum#5004

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants