Copilot commented on code in PR #839:
URL: https://github.com/apache/iceberg-cpp/pull/839#discussion_r3629395223
##########
.github/workflows/test.yml:
##########
@@ -25,6 +25,11 @@ on:
- '**'
pull_request:
types: [opened, synchronize, reopened, ready_for_review]
+ paths-ignore:
+ - '.devcontainer/**'
+ - '**/*.md'
+ - 'dev/**'
+ - 'mkdocs/**'
Review Comment:
The PR description says PR checks should be skipped for \"Markdown,
documentation, devcontainer, and release-development-only changes\", but the
implemented `paths-ignore` list only covers `.devcontainer/**`, `**/*.md`,
`dev/**`, and `mkdocs/**`. If documentation or release/dev-only content lives
outside these patterns (e.g., `docs/**`, `site/**`, `CHANGELOG*`, images under
a docs directory, or other doc extensions like `.rst`/`.adoc`/`.mdx`), the
workflows will still run on those PRs. Consider either expanding `paths-ignore`
to match the repo’s actual documentation/release-dev-only paths (and
extensions), or updating the PR description to reflect the narrower scope being
applied.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
