Skip to content

Commit 454d747

Browse files
EdwardAngertClaude
and
Claude
committed
refactor: use existing file detection for Vale
- Replace tj-actions/changed-files with existing file detection - Use the changed_files_csv output from docs-core-setup step - Fix 'Action not found' error by removing unreliable dependency - Maintain performance optimization by only checking changed files 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 2760742 commit 454d747

File tree

1 file changed

+15
-10
lines changed

1 file changed

+15
-10
lines changed

.github/workflows/docs-unified.yaml

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -127,20 +127,25 @@ jobs:
127127
128128
# 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: Get changed files for Vale
131-
id: changed-files
132-
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
130+
- name: Extract changed files for Vale
131+
id: extract-changed-files
132+
if: steps.docs-core-setup.outputs.needs_vale == 'true' && steps.docs-core-setup.outputs.has_changes == 'true'
133+
shell: bash
134+
run: |
135+
# Use the CSV of changed files detected by docs-core
136+
CHANGED_FILES="${{ steps.docs-core-setup.outputs.changed_files_csv }}"
137+
if [ -n "$CHANGED_FILES" ]; then
138+
echo "Files to check: $CHANGED_FILES"
139+
echo "files_to_check=$CHANGED_FILES" >> $GITHUB_OUTPUT
140+
else
141+
echo "No files to check"
142+
fi
138143
139144
- name: Run Vale style checks
140-
if: steps.docs-core-setup.outputs.needs_vale == 'true' && steps.changed-files.outputs.all_changed_files != ''
145+
if: steps.docs-core-setup.outputs.needs_vale == 'true' && steps.extract-changed-files.outputs.files_to_check != ''
141146
uses: errata-ai/vale-action@v2.1.1
142147
with:
143-
files: ${{ steps.changed-files.outputs.all_changed_files }}
148+
files: ${{ steps.extract-changed-files.outputs.files_to_check }}
144149
reporter: github-check
145150
fail_on_error: false
146151
vale_flags: "--config=.github/docs/vale/.vale.ini"

0 commit comments

Comments
 (0)