[GitHub] [airflow] jstern commented on issue #7133: [AIRFLOW-6535] add exception to fail without retry

[GitBox <gi...@apache.org>]

jstern commented on issue #7133: [AIRFLOW-6535] add exception to fail without retry
URL: https://github.com/apache/airflow/pull/7133#issuecomment-605339729
 
 
   Thanks @KevinYang21 - I definitely would like to see it get merged, but I'm stuck on a strange problem with the docs build throwing warnings due to the problem I described in https://github.com/apache/airflow/pull/7133#discussion_r389372194 above. 
   
   The short version of the issue is that my PR adds `airflow.exceptions` to the API docs, but this causes warnings in the doc build because `AirflowException` is already included (incorrectly) in the yandex provider API docs. I have not yet had the time or inspiration to figure out why that bug exists in master or how to correct it.
   
   Another option that would let the build succeed would be to revert the change that adds the exceptions module to the api docs, but since I believe the docs in master are wrong I have been reluctant to take this step.
   
   Perhaps I should file a JIRA ticket for the docs bug in master and hope someone who understands Sphinx and autoapi better than I do knows how to fix it?

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


With regards,
Apache Git Services