kevinjqliu commented on code in PR #3832:
URL: https://github.com/apache/iceberg-python/pull/3832#discussion_r3917184433
##########
.github/workflows/python-ci.yml:
##########
@@ -24,19 +24,7 @@ on:
branches:
- 'main'
pull_request:
- paths:
- - '**' # Include all files and directories in
the repository by default.
- - '!.github/workflows/**' # Exclude all workflow files
- - '.github/workflows/python-ci.yml' # except the current file.
- - '!.github/ISSUE_TEMPLATE/**' # Exclude files and directories that
don't impact tests or code like templates, metadata, and documentation.
- - '!.gitignore'
- - '!.asf.yml'
- - '!mkdocs/**'
- - '!.gitattributes'
- - '!README.md'
- - '!CONTRIBUTING.md'
- - '!LICENSE'
- - '!NOTICE'
Review Comment:
i think this is a ok trade off to enable merge queue.
merge queue requires checks to always run
--
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]
