SEZ9 commented on PR #12173: URL: https://github.com/apache/seatunnel/pull/12173#issuecomment-5902739559
@Rangsh — yes, holding the head at `7948e2640` with no further pushes is exactly the right call. Nothing is needed from you on F1–F6: they were closed on that head, and the docs concern behind the earlier `CHANGES_REQUESTED` review was already addressed in `9a6077c58` (the current head no longer touches `docs/en` / `docs/zh`), so a new commit would only reset the confirmation we already have. To answer your comment directly: there is nothing left on your side. The only remaining step is clearing that stale review state so `reviewDecision` moves off `CHANGES_REQUESTED` and the PR is no longer `BLOCKED`; that requires a re-review or dismissal by someone with write access, not a code change from you. If that re-review does ask for anything, please push it as a new commit on top of `7948e2640` rather than force-pushing, so F1–F6 can be re-confirmed against a clean delta. <!-- streview-comment:1411 --> -- 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]
