Laffery commented on PR #28593:
URL: https://github.com/apache/flink/pull/28593#issuecomment-4852367850
@MartijnVisser
Thanks for the upgrade work. One thing I think we should clean up before
merging: after removing the lint-staged setup, quite a few migrated templates
appear to have escaped formatting.
For example, the new Angular control-flow blocks in
job-rescales.component.html, job-overview-drawer-subtasks.component.html, and
submit.component.html contain unindented bodies, chained `} @if (...) {`
blocks, and split `@for` expressions that make the templates harder to review
and maintain.
Could you please run `npm run lint:fix` to format the whole project?
--
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]