Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds path filters to GitHub Actions workflows to prevent unnecessary CI/CD runs, particularly helping Dependabot PRs pass CI more efficiently. By adding path filters, workflows will only trigger when relevant files are modified.
Changes:
- Added path filters to 9 workflow files to limit when they trigger based on file changes
- Added concurrency control to json-format-check.yml and csharpier-autofix.yml
- Added workflow_dispatch trigger to dotnet-tests.yml for manual execution
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/yaml-format-lint.yml | Added path filters for YAML files and related configuration |
| .github/workflows/prettier-autofix.yml | Added path filters for all file types that Prettier formats |
| .github/workflows/markdown-link-validity.yml | Added path filters for markdown files and lychee configuration |
| .github/workflows/markdown-link-text-check.yml | Added path filters for markdown files and Python validation scripts |
| .github/workflows/markdown-json.yml | Added path filters for markdown, JSON, and related configuration |
| .github/workflows/lint-doc-links.yml | Added path filters for markdown, docs, and linting scripts |
| .github/workflows/json-format-check.yml | Added path filters for JSON files, concurrency control, and configuration |
| .github/workflows/dotnet-tests.yml | Added path filters for .NET files and workflow_dispatch trigger |
| .github/workflows/csharpier-autofix.yml | Added path filters for C# files and dotnet-tools configuration |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated 3 comments.
Comments suppressed due to low confidence (5)
.github/workflows/markdown-link-validity.yml:24
- This workflow is missing required elements according to the new workflow consistency skill that's being introduced in this PR. It should include:
- A
concurrencyblock withcancel-in-progress: true(before thejobssection) - An explicit
permissionsblock (even if justcontents: read) - A
timeout-minuteson the job persist-credentials: falseon the checkout step
All other workflows in this PR are being updated with these elements, but this one was only partially updated with path filters.
jobs:
link-check:
runs-on: ubuntu-latest
steps:
.github/workflows/markdown-link-text-check.yml:24
- This workflow is missing required elements according to the new workflow consistency skill. It should include:
- A
concurrencyblock withcancel-in-progress: true(placed betweenonandjobs) - An explicit
permissionsblock (even if justcontents: read) - A
timeout-minuteson the job persist-credentials: falseon the checkout step
All other workflows in this PR are being updated with these elements, but this one was only partially updated with path filters.
jobs:
check:
runs-on: ubuntu-latest
steps:
.github/workflows/lint-doc-links.yml:26
- This workflow is missing required elements according to the new workflow consistency skill. It should include:
- A
concurrencyblock withcancel-in-progress: true(placed betweenonandjobs) - An explicit
permissionsblock (even if justcontents: read) - A
timeout-minuteson the job persist-credentials: falseon the checkout step
All other workflows in this PR are being updated with these elements, but this one was only partially updated with path filters.
jobs:
lint:
runs-on: ubuntu-latest
steps:
.github/workflows/markdown-json.yml:40
- This workflow is missing required elements according to the new workflow consistency skill. It should include:
- A
concurrencyblock withcancel-in-progress: true(placed betweenonandpermissions) - A
timeout-minuteson the job persist-credentials: falseon the checkout step- The single quotes around node-version and cache should be changed to double quotes for Prettier consistency
The property order should also be: name → on → concurrency → permissions → jobs (currently permissions comes before concurrency should be added).
permissions:
contents: read
jobs:
validate:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v6
.github/workflows/dotnet-tests.yml:28
- This workflow is missing required elements according to the new workflow consistency skill. It should include:
- A
concurrencyblock withcancel-in-progress: true(placed betweenonandpermissions) - A
timeout-minuteson the job persist-credentials: falseon the checkout step
The property order should also be: name → on → concurrency → permissions → jobs.
jobs:
test:
runs-on: ubuntu-latest
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Description
These changes might help dependabot pass CI/CD
Related Issue
Type of Change
Checklist