mirror of
https://github.com/Kitware/CMake.git
synced 2025-10-19 19:43:23 +08:00
Utilities/Sphinx: Don't ignore flake8 E402
Refactor commit 1f39a3cd1a
(Utilities/Sphinx: Restore explicit check for
Sphinx 2.x or later) to avoid needing to suppress flake8 E402. While
ignoring it with respect to the docutils/sphinx imports and the sphinx
version check was correct, the need to disable it for the whole file was
suboptimal.
This commit is contained in:
@@ -1,3 +0,0 @@
|
|||||||
[flake8]
|
|
||||||
per-file-ignores =
|
|
||||||
cmake.py: E402
|
|
@@ -11,24 +11,24 @@ from typing import Any, List, Tuple, Type, cast
|
|||||||
|
|
||||||
import sphinx
|
import sphinx
|
||||||
|
|
||||||
# Require at least Sphinx 2.x.
|
# The following imports may fail if we don't have Sphinx 2.x or later.
|
||||||
# flake8 issues E402 for imports after this, but the purpose of this
|
if sphinx.version_info >= (2,):
|
||||||
# check is to fail more clearly if the imports below will fail.
|
from docutils.utils.code_analyzer import Lexer, LexerError
|
||||||
assert sphinx.version_info >= (2,)
|
from docutils.parsers.rst import Directive, directives
|
||||||
|
from docutils.transforms import Transform
|
||||||
|
from docutils.nodes import Element, Node, TextElement, system_message
|
||||||
|
from docutils import io, nodes
|
||||||
|
|
||||||
from docutils.utils.code_analyzer import Lexer, LexerError
|
from sphinx.directives import ObjectDescription, nl_escape_re
|
||||||
from docutils.parsers.rst import Directive, directives
|
from sphinx.domains import Domain, ObjType
|
||||||
from docutils.transforms import Transform
|
from sphinx.roles import XRefRole
|
||||||
from docutils.nodes import Element, Node, TextElement, system_message
|
from sphinx.util.docutils import ReferenceRole
|
||||||
from docutils import io, nodes
|
from sphinx.util.nodes import make_refnode
|
||||||
|
from sphinx.util import logging, ws_re
|
||||||
from sphinx.directives import ObjectDescription, nl_escape_re
|
from sphinx import addnodes
|
||||||
from sphinx.domains import Domain, ObjType
|
else:
|
||||||
from sphinx.roles import XRefRole
|
# Sphinx 2.x is required.
|
||||||
from sphinx.util.docutils import ReferenceRole
|
assert sphinx.version_info >= (2,)
|
||||||
from sphinx.util.nodes import make_refnode
|
|
||||||
from sphinx.util import logging, ws_re
|
|
||||||
from sphinx import addnodes
|
|
||||||
|
|
||||||
# END imports
|
# END imports
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user