Skip to content

Commit 2760742

Browse files
EdwardAngertClaude
and
Claude
committed
perf: optimize Vale to only check changed files
- Use tj-actions/changed-files to get only modified markdown files - Pass only changed files to Vale action instead of entire docs directory - Fix performance issue causing Vale to run for 20+ minutes - Skip Vale check if no markdown files have changed 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent e55e3c7 commit 2760742

File tree

1 file changed

+12
-3
lines changed

1 file changed

+12
-3
lines changed

.github/workflows/docs-unified.yaml

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -125,13 +125,22 @@ jobs:
125125
./scripts/pnpm_install.sh --prefer-offline || ./scripts/pnpm_install.sh --no-frozen-lockfile
126126
echo "Node.js dependencies installed successfully"
127127
128-
# Vale style checking for documentation
128+
# Vale style checking for documentation - only checks changed files
129129
# This is the ONLY place Vale is actually run - the action in docs-core is a placeholder
130-
- name: Run Vale style checks
130+
- name: Get changed files for Vale
131+
id: changed-files
131132
if: steps.docs-core-setup.outputs.needs_vale == 'true'
133+
uses: tj-actions/changed-files@2d756ea4c53f7f6b37fbf1ca142f1614907ee512 # v41
134+
with:
135+
files: |
136+
docs/**/*.md
137+
**/*.md
138+
139+
- name: Run Vale style checks
140+
if: steps.docs-core-setup.outputs.needs_vale == 'true' && steps.changed-files.outputs.all_changed_files != ''
132141
uses: errata-ai/vale-action@v2.1.1
133142
with:
134-
files: docs
143+
files: ${{ steps.changed-files.outputs.all_changed_files }}
135144
reporter: github-check
136145
fail_on_error: false
137146
vale_flags: "--config=.github/docs/vale/.vale.ini"

0 commit comments

Comments
 (0)