kbendick commented on a change in pull request #4210:
URL: https://github.com/apache/iceberg/pull/4210#discussion_r814304434
##########
File path: .github/workflows/flink-ci.yml
##########
@@ -19,27 +19,13 @@
name: "Flink CI"
on:
- push:
Review comment:
I'm going to remove the `paths-ignore`, as when I spoke to @rdblue about
this a while ago, the ideal situation was to run all tests after merging to
master.
That way, we don't have to keep both lists synchronized. Then eventually we
can have a separate PR workflow just for merging to master and for tagging (new
releases).
--
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]
