Skip to content

Limit blueprint workflow to docs changes#4

Merged
Robertboy18 merged 1 commit into
mainfrom
blueprint-path-filter
Jun 3, 2026
Merged

Limit blueprint workflow to docs changes#4
Robertboy18 merged 1 commit into
mainfrom
blueprint-path-filter

Conversation

@Robertboy18

Copy link
Copy Markdown
Member

Avoid running the expensive blueprint/docs/site build for PRs that only touch unrelated code. The workflow still runs for docs/site inputs, dependency pins, README, and workflow changes.

Copilot AI review requested due to automatic review settings June 3, 2026 17:27
@Robertboy18 Robertboy18 merged commit a19506e into main Jun 3, 2026
1 check passed
@Robertboy18 Robertboy18 deleted the blueprint-path-filter branch June 3, 2026 17:28

Copilot AI left a comment

Copy link
Copy Markdown

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 updates the GitHub Actions blueprint workflow trigger conditions to avoid running the expensive docs/site build when changes don’t affect docs-related inputs.

Changes:

  • Add paths: filters to push on main for docs/site-related files and dependency pins.
  • Add the same paths: filters to pull_request targeting main to reduce workflow runs on unrelated PR changes.

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

Comment on lines 4 to +15
push:
branches: ["main"]
paths:
- ".github/workflows/blueprint.yml"
- "blueprint/**"
- "home_page/**"
- "scripts/docs/**"
- "scripts/checks/dependency_audit.py"
- "lakefile.lean"
- "lake-manifest.json"
- "lean-toolchain"
- "README.md"
Comment on lines 16 to +27
pull_request:
branches: ["main"]
paths:
- ".github/workflows/blueprint.yml"
- "blueprint/**"
- "home_page/**"
- "scripts/docs/**"
- "scripts/checks/dependency_audit.py"
- "lakefile.lean"
- "lake-manifest.json"
- "lean-toolchain"
- "README.md"
Comment on lines +6 to +10
paths:
- ".github/workflows/blueprint.yml"
- "blueprint/**"
- "home_page/**"
- "scripts/docs/**"
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.

2 participants