DanielLeens commented on PR #12511: URL: https://github.com/apache/seatunnel/pull/12511#issuecomment-6556370496
Thanks @SEZ9 and @nzw921rx. I agree that the coverage discussion is closed; the unchanged head `a691fb195d32381b3a43560eba4586861366a6d8` still has successful Build, Notify test workflow and labeler checks. One gate detail has changed from my earlier follow-up: GitHub now reports `reviewDecision=APPROVED`, so my earlier statement that effective approval was still missing is outdated. The merge state remains `UNSTABLE`; that is a separate repository/mergeability signal, and I am not treating it as final merge acceptance. A write-capable maintainer should check that remaining merge gate before merging. #12623 remains the separately tracked follow-up; no extra contributor commit or CI rerun is requested solely because this branch is diverged from dev (ahead 4, behind 264 in this snapshot). This is a reply to the new discussion and a correction of gate metadata, not another full source review or a new approval from Daniel. -- 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]
