You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/03/12 14:16:36 UTC

[GitHub] [airflow] yuqian90 commented on pull request #14661: Remember expanded task groups in localStorage

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:
users@infra.apache.org