pierrejeambrun commented on code in PR #47442:
URL: https://github.com/apache/airflow/pull/47442#discussion_r1983091787
##########
airflow/ui/src/pages/Dag/Code/Code.tsx:
##########
@@ -73,11 +86,35 @@ export const Code = () => {
return (
<Box>
<HStack justifyContent="space-between" mt={2}>
- {dag?.last_parsed_time !== undefined && (
- <Heading as="h4" fontSize="14px" size="md">
- Parsed at: <Time datetime={dag.last_parsed_time} />
- </Heading>
- )}
+ <HStack gap={5}>
+ {dag?.last_parsed_time !== undefined && (
+ <Heading as="h4" fontSize="14px" size="md">
+ Parsed at: <Time datetime={dag.last_parsed_time} />
+ </Heading>
+ )}
+
+ {
+ // eslint-disable-next-line
@typescript-eslint/prefer-optional-chain
Review Comment:
This is necessary because the ts eslint plugin produces a wrong logic here.
This has been fixed, I suspect we need to upgrade the lib.
--
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]
