DanielLeens commented on PR #12023: URL: https://github.com/apache/seatunnel/pull/12023#issuecomment-5662000019
Thanks for the second pair of eyes, @nzw921rx. To recap for the record: my own conclusion on this PR (head `27ee284b`) is still **Ready to merge**, with only three non-blocking follow-up recommendations (the `Fixes #10807` auto-close scope, the silent no-op guards, and a shared-config test-isolation nit) — none of them hold up merging. CI on this head is green (`Build: SUCCESS`, https://github.com/apache/seatunnel/runs/99524505215). I only have comment rights here, so a write-capable maintainer will need to action the actual merge. -- 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]
