Skip to content

ci(pr-check): expand path filters across pr-check workflows #44657

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
caugner merged 4 commits into from
Jul 17, 2026
Merged

ci(pr-check): expand path filters across pr-check workflows #44657

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
0d21160
ci(pr-check): expand path filters across pr-check workflows
caugner Jul 7, 2026
ce04ded
style(prettier): format scripts
caugner Jul 7, 2026
8700c84
ci(pr-check_yml): run when root YAML files change
caugner Jul 15, 2026
b83fda5
ci(pr-check_url-issues): add additional paths
caugner Jul 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .github/workflows/pr-check_cspell_lists.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,10 @@ name: Check cSpell lists
on:
pull_request:
paths:
- .github/workflows/pr-check_cspell_lists.yml
- .nvmrc
- .vscode/dictionaries/*
- scripts/sort_and_unique_file_lines.js

# No GITHUB_TOKEN permissions, as we don't use it.
permissions: {}
Expand Down
6 changes: 5 additions & 1 deletion .github/workflows/pr-check_javascript.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,14 @@ name: JavaScript lint
on:
pull_request:
paths:
- .github/workflows/pr-check_javascript.yml
- .nvmrc
- package.json
- package-lock.json
- "*.js"
- "*.mjs"
- "**/*.js"
- "**/*.mjs"
- .github/workflows/pr-check_javascript.yml

# No GITHUB_TOKEN permissions, as we only use it to increase API limit.
permissions: {}
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/pr-check_json.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,12 @@ name: JSON lint
on:
pull_request:
paths:
- .github/workflows/pr-check_json.yml
- .nvmrc
- "*.json"
- "*.jsonc"
- "**/*.json"
- "**/*.jsonc"
- .github/workflows/pr-check_json.yml

# No GITHUB_TOKEN permissions, as we only use it to increase API limit.
permissions: {}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pr-check_scripts.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@ name: Check scripts
on:
pull_request:
paths:
- .github/workflows/pr-check_scripts.yml
- .nvmrc
- package.json
- package-lock.json
- .github/workflows/pr-check_scripts.yml

# No GITHUB_TOKEN permissions, as we only use it to increase API limit.
permissions: {}
Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/pr-check_url-issues.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,12 @@ name: Check URL issues
on:
pull_request:
paths:
- .github/workflows/pr-check_url-issues.yml
- .nvmrc
- package.json
- package-lock.json
- "files/**/*.md"
- scripts/log-url-issues.js

# No GITHUB_TOKEN permissions, as we don't use it.
permissions: {}
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/pr-check_yml.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,12 @@ name: Lint YAML
on:
pull_request:
paths:
- .github/workflows/pr-check_yml.yml
- .nvmrc
- package.json
- package-lock.json
- "*.yml"
- "**/*.yml"
Comment thread
caugner marked this conversation as resolved.
- .github/workflows/pr-check_yml.yml

# No GITHUB_TOKEN permissions, as we only use it to increase API limit.
permissions: {}
Expand Down
2 changes: 1 addition & 1 deletion scripts/sort_and_unique_file_lines.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ for (const filePath of filePaths) {
return a < b ? -1 : a > b ? 1 : 0;
});

for (let i = 0; i < lines.length; ) {
for (let i = 0; i < lines.length;) {
const line = lines[i];
if (line.trim() !== "") {
uniq.push(line);
Expand Down