From 5d261446645132280b53ee180e07ff9365c239a8 Mon Sep 17 00:00:00 2001 From: Nguyen Van Trung Date: Wed, 30 May 2018 10:23:15 +0700 Subject: [PATCH] Remove ignore D000 in validation code This commit going to remove --ignore D000 which had raised alot of issues, such as: - "D000 Cannot analyze code. Pygments package not found." - Cannot check LOG which have '_' symbol in code-base. And they also fixed in https://review.openstack.org/#/c/568729/ Change-Id: Ic5258e0262c11b9f878d867a3354aa7d1f6e67b4 --- doc/source/admin/writing_pipeline_stages.rst | 4 +--- test-requirements.txt | 1 + tox.ini | 5 +---- 3 files changed, 3 insertions(+), 7 deletions(-) diff --git a/doc/source/admin/writing_pipeline_stages.rst b/doc/source/admin/writing_pipeline_stages.rst index 55f55e105..e838beaa3 100644 --- a/doc/source/admin/writing_pipeline_stages.rst +++ b/doc/source/admin/writing_pipeline_stages.rst @@ -75,9 +75,7 @@ This is an example of small package with a stage that can process queue-related requests in Zaqar. The stage does not do anything useful, but is good as example. -File tree structure of the package: - -.. code-block:: none +File tree structure of the package:: . ├── setup.py diff --git a/test-requirements.txt b/test-requirements.txt index 2534cf3b6..99d6442ac 100644 --- a/test-requirements.txt +++ b/test-requirements.txt @@ -18,6 +18,7 @@ PyMySQL>=0.7.6 # MIT License coverage!=4.4,>=4.0 # Apache-2.0 ddt>=1.0.1 # MIT doc8>=0.6.0 # Apache-2.0 +Pygments>=2.2.0 # BSD license fixtures>=3.0.0 # Apache-2.0/BSD testscenarios>=0.4 # Apache-2.0/BSD diff --git a/tox.ini b/tox.ini index b57dddf35..65153ef77 100644 --- a/tox.ini +++ b/tox.ini @@ -46,6 +46,7 @@ commands = {posargs} [testenv:docs] deps = -r{toxinidir}/doc/requirements.txt + -r{toxinidir}/test-requirements.txt commands = doc8 doc/source @@ -83,10 +84,6 @@ extensions = .rst, .yaml # Maximal line length should be 80 but we have some overlong lines. # Let's not get far more in. max-line-length = 80 -# Disable some doc8 checks: -# D000: Check RST validity -# - cannot handle "none" for code-block directive -ignore = D000 [hacking] local-check-factory = zaqar.hacking.checks.factory