DanielLeens commented on PR #11982: URL: https://github.com/apache/seatunnel/pull/11982#issuecomment-5451406653
Follow-up on CI status: the `Build` check (which includes `Code style`) is now `SUCCESS` on the exact commit I reviewed (`72abed17`) — see https://github.com/apache/seatunnel/runs/98401765756. The head hasn't moved since my last review, so this closes out the one open item ("please make sure it finishes green before merging"). With CI green and no source-side blockers remaining, my conclusion stands as **Ready to merge**. The one item I flagged (Issue 1, missing negative test coverage for `withDiagnostics=false` on `/running-jobs`) is non-blocking and can be picked up as a fast-follow. Thanks again for the thorough round! -- 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]
