mik-laj commented on a change in pull request #4894: [AIRFLOW-3996] Add view source link to included fragments URL: https://github.com/apache/airflow/pull/4894#discussion_r265782625
########## File path: docs/exts/exampleinclude.py ########## @@ -0,0 +1,189 @@ +from os import path + +from docutils import nodes +from docutils.parsers.rst import directives +from six import text_type +from sphinx import addnodes +from sphinx.directives.code import LiteralIncludeReader +from sphinx.locale import _ +from sphinx.pycode import ModuleAnalyzer +from sphinx.util import logging +from sphinx.util import parselinenos +from sphinx.util.docutils import SphinxDirective +from sphinx.util.nodes import set_source_info + +logger = logging.getLogger(__name__) + + +class example_header(nodes.reference, nodes.FixedTextElement): + pass + + +class ExampleInclude(SphinxDirective): Review comment: I based on: https://github.com/sphinx-doc/sphinx/blob/v1.8.3/sphinx/directives/code.py I've automatically formatted the code to be compatible with my company's requirements. We use black. Changes from my team are undergoing internal review. During it, we had a lot of discussion about style, which forced us to introduce automatic formatting of the code. I made changes in the `option_spec` field. If I extend this class then I will have to make identical modifications programmatically. It is worth noting that the value of this field is closely related to the implementation. If the value of the field changes, then the implementation must also be changed. ---------------------------------------------------------------- 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] With regards, Apache Git Services
