X-Git-Url: https://gerrit.fd.io/r/gitweb?p=csit.git;a=blobdiff_plain;f=tox.ini;h=2084e5abdfa242e12969230d39f923870a3721a3;hp=566343bcf40d579cd89886c1b2a20da72df2a807;hb=HEAD;hpb=8609df3a0e9f5510a5485d100ed03a40b051c199 diff --git a/tox.ini b/tox.ini index 566343bcf4..e0ac85c22a 100644 --- a/tox.ini +++ b/tox.ini @@ -1,4 +1,4 @@ -# Copyright (c) 2023 Cisco and/or its affiliates. +# Copyright (c) 2024 Cisco and/or its affiliates. # Licensed under the Apache License, Version 2.0 (the "License"); # you may not use this file except in compliance with the License. # You may obtain a copy of the License at: @@ -26,16 +26,14 @@ [tox] # Fast and brief checkers to front, slow or verbose checkers to back. -envlist = job_spec, copyright_year, gpl_license, - new_line_length, line_length, autogen, pylint, doc_verify +envlist = copyright_year, gpl_license, new_line_length, line_length, autogen, + pylint # The following is needed as tox requires setup.py by default. skipsdist = true # Just a shorthand to avoid long lines. script_dir = ./resources/libraries/bash/entry/tox -# TODO: Tox prints various warnings. Figure them out and fix them. - # Recommended ordering withinn testenv definition: # + General comments. # + ignore_outcome for non-voting checkers, @@ -44,31 +42,27 @@ script_dir = ./resources/libraries/bash/entry/tox # + whitelist_externals to enable bash. # + Commands with a single bash to launch the corresponding script. -# Keep testenvs sorted alphabetically, please. - [testenv:autogen] setenv = PYTHONPATH = {toxinidir} -whitelist_externals = /bin/bash +whitelist_externals = bash commands = bash {[tox]script_dir}/autogen.sh [testenv:copyright_year] -whitelist_externals = /bin/bash +whitelist_externals = bash commands = bash {[tox]script_dir}/copyright_year.sh -[testenv:doc_verify] -# Fix all documentation error before enabling voting. -ignore_outcome = true -whitelist_externals = /bin/bash -commands = bash {[tox]script_dir}/doc_verify.sh +[testenv:fix_copyright_year] +whitelist_externals = bash +commands = bash {[tox]script_dir}/fix_copyright_year.sh [testenv:gpl_license] -whitelist_externals = /bin/bash +whitelist_externals = bash commands = bash {[tox]script_dir}/gpl_license.sh [testenv:line_length] # Fix all transgressions and remove the ignore_outcome flag. ignore_outcome = true -whitelist_externals = /bin/bash +whitelist_externals = bash commands = bash {[tox]script_dir}/line.sh [testenv:new_line_length] @@ -76,7 +70,7 @@ commands = bash {[tox]script_dir}/line.sh # As current implementation is too dumb to filter those out, # the checker has to remain non-voting. ignore_outcome = true -whitelist_externals = /bin/bash +whitelist_externals = bash commands = bash {[tox]script_dir}/new_line.sh [testenv:pylint] @@ -84,22 +78,6 @@ commands = bash {[tox]script_dir}/new_line.sh ignore_outcome = true setenv = PYTHONPATH = {toxinidir} basepython = python3 -deps = - pylint==2.4.3 - -r ./requirements.txt -whitelist_externals = /bin/bash +deps = -r ./requirements.txt +whitelist_externals = bash commands = bash {[tox]script_dir}/pylint.sh - -# Keep testenvs sorted alphabetically, please. - -# TODO: Add a checker against unresolved merge conflicts. -# See: https://gerrit.fd.io/r/c/vpp/+/26956 -# TODO: Create voting "pylint violations should not increase" checker. -# TODO: Create voting checker to reject suites with Force Tags of other suite. -# TODO: Create voting checker against tags not in tag_documentation.rst file. -# TODO: Create Robot suite Documentation checker (backslash if not next mark). -# TODO: Create .yaml specific checker, so people can override long line check. -# TODO: Create .rst specific checker, if there is one allowing -# to override line check. -# TODO: You get the idea, replace line checker with something smarter -# wherever possible.