yuqian90 commented on pull request #14661:
URL: https://github.com/apache/airflow/pull/14661#issuecomment-797517006


   > @yuqian90 this is a nice feature, thank you!
   > 
   > We're in the process of migrating all of our JavaScript to external files 
(#14115) so we can integrate linting in our pre-commit hooks (#14154). There is 
some syntax currently that doesn't align with our current config. Would you 
mind updating those (left comments inline) to save us some time with these 
other efforts? (please double-check that my suggestions are indeed correct)
   
   Thanks for the detailed comments. I corrected these syntax issues. That 
said, I did not do a thorough check of the entire graph.html to look for other 
syntax issues to avoid distracting the objective of this PR.


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to