DanielLeens commented on PR #12725:
URL: https://github.com/apache/seatunnel/pull/12725#issuecomment-6122334558

   Thanks for the update, @goutamadwant, and for marking the PR ready for 
review!
   
   I rechecked the description against my last review (head `24dc76809`, 
unchanged): the "Post-merge check" line under Behaviour differences now covers 
my only remaining Low item (verify the next fork PR gets labels plus `Build`, 
and revert if not, since the old `pull_request_target` workflows keep working 
until the November cutoff). So I do not see any open source-side blocker from 
my side.
   
   One CI fact in my previous review has also changed: I said the apache-side 
`Build` check was still in progress. It is now green, together with `Signal 
pull request update`, `labeler` and `Notify test workflow`. The only remaining 
gate is the committer approval (`REVIEW_REQUIRED`), which a maintainer with 
write access needs to give; my comment does not replace that.
   


-- 
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]

Reply via email to