X-Git-Url: https://gerrit.fd.io/r/gitweb?p=csit.git;a=blobdiff_plain;f=tox.ini;h=4617e73e54bdc4375a00caad550ccbaa5933da66;hp=126af2d454267a767e3909a80bf7b9952e3386e1;hb=HEAD;hpb=9e22f65d5e9f7045625a8d6bf79ce75d836e1447 diff --git a/tox.ini b/tox.ini index 126af2d454..e0ac85c22a 100644 --- a/tox.ini +++ b/tox.ini @@ -1,4 +1,4 @@ -# Copyright (c) 2022 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, model_version, - 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,43 +42,35 @@ 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:model_version] -whitelist_externals = /bin/bash -commands = bash {[tox]script_dir}/model_version.sh - [testenv:new_line_length] # Adding .svg, big .rst tables, .yaml and similar still requires long lines. # 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] @@ -88,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.