liurenjie1024 commented on code in PR #1693:
URL: https://github.com/apache/iceberg-rust/pull/1693#discussion_r2375220756
##########
.github/workflows/bindings_python_ci.yml:
##########
@@ -23,10 +23,25 @@ on:
- main
pull_request:
paths:
- - ".github/workflows/bindings_python_ci.yml"
- - "bindings/python/**"
- - "crates/iceberg/**"
- - "crates/integrations/datafusion/**"
+ - '**' # Include all files and
directories in the repository by default.
+ - '!.github/workflows/**' # Exclude all workflow files
Review Comment:
I don't think it makes sense to exclude workflow changes, which may cause
confusions. We should be careful when excluding files.
--
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]
