DanielLeens commented on PR #13008:
URL: https://github.com/apache/seatunnel/pull/13008#issuecomment-6461939085

   Follow-up at head `e86481f35`: the three UI blockers from the first review 
are addressed. The monitoring table now renders only text-typed fields and 
preserves the literal `fixed` column value; the jobs test mock retains the real 
`requestFailureMessage` export; and the managers tests assert both master 
(1200) and worker (1460) scroll widths. Scoped Spotless, Prettier, and `git 
diff --check` passed. Local compilation and tests were not run under the 
SeaTunnel validation policy. The [current-head fork 
Build](https://github.com/DanielLeens/seatunnel/actions/runs/40641069202) is 
pending, so this is ready for human code review, **not merge-ready**. The 
previously noted low-priority status-port and invalid-mode message concerns 
remain for reviewer judgment.


-- 
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]

Reply via email to