DanielLeens commented on PR #12242: URL: https://github.com/apache/seatunnel/pull/12242#issuecomment-5652817369
Confirmed: the fork CI for the current head (`014fd63`) is now fully green (Build, License header, Code style, Dead links, Sanity check, Check Helm Chart all pass) — the queued-jobs caveat I noted at the end of my last review is resolved, and branch protection should be satisfied. My conclusion from the last round stands: Ready to merge, no outstanding blockers on my side. Thanks for the approval, davidzollo — since my account here is comment-only and can't merge, I'll leave the actual merge to you or another committer with write access whenever convenient. Nice work carrying this all the way through, goutamadwant — the wire-level HTTP assertions and the MinIO-backed ETag/content/count-unchanged proof are genuinely strong evidence for the "does not touch data" claim, not just a docstring promise. -- 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]
