potiuk commented on a change in pull request #20951:
URL: https://github.com/apache/airflow/pull/20951#discussion_r788704940
##########
File path: setup.cfg
##########
@@ -92,12 +92,6 @@ install_requires =
cryptography>=0.9.3
deprecated>=1.2.13
dill>=0.2.2, <0.4
- # Sphinx RTD theme 0.5.2. introduced limitation to docutils to account for
some docutils markup
- # change:
- # https://github.com/readthedocs/sphinx_rtd_theme/issues/1112
- # This limitation can be removed after this issue is closed:
- # https://github.com/readthedocs/sphinx_rtd_theme/issues/1115
- docutils<0.17
Review comment:
Agree. This ain't easy thing to do. But possibly making eager-update
take into account the extras in "one pass" might be possible @uranusjr ? Not
sure if that's possible, or if there are some reasons for that. I could
possibly open an issue for that, but I am afraid my issues might not be too
welcomed as my persistency in improving warning message in the `root` case is
seen as a bit `toxic` (even if it is a real issue to tackle).
--
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]
