From 52bcd6f7d9cce360cd4614c794d1a2f09751817d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Felipe=20Corr=C3=AAa=20da=20Silva=20Sanches?= Date: Wed, 16 Feb 2022 05:03:53 -0300 Subject: [PATCH] fixing the GitHub Actions build addressing linter warnings/errors --- Lib/languages/lang_support.py | 12 ++++++++---- tox.ini | 7 +++---- 2 files changed, 11 insertions(+), 8 deletions(-) diff --git a/Lib/languages/lang_support.py b/Lib/languages/lang_support.py index df7a6bbe4..a370a24ae 100644 --- a/Lib/languages/lang_support.py +++ b/Lib/languages/lang_support.py @@ -14,17 +14,21 @@ # See the License for the specific language governing permissions and # limitations under the License. # -"""Helper API for interaction with languages/regions/scripts data on the Google Fonts collection. """ +Helper API for interaction with languages/regions/scripts +data on the Google Fonts collection. +""" + +import glob +import os from fontTools.ttLib import TTFont from google.protobuf import text_format from hyperglot import parse as hyperglot_parse -from languages import fonts_public_pb2 from pkg_resources import resource_filename -import os -import glob +from languages import fonts_public_pb2 + DATA_DIR = resource_filename("languages", "data") diff --git a/tox.ini b/tox.ini index ea143639a..3418ce5af 100644 --- a/tox.ini +++ b/tox.ini @@ -19,7 +19,6 @@ passenv = LD_LIBRARY_PATH LD_PRELOAD deps = coverage skip_install = true commands = - coverage combine coverage report coverage html @@ -40,8 +39,8 @@ deps = sphinx sphinx_rtd_theme recommonmark -wont_fix = -maybe_someday = fixme,missing-docstring +wont_fix = invalid-name,protected-access,too-many-lines +maybe_someday = fixme,missing-docstring,line-too-long commands = pylint --disable={[testenv:pylint]wont_fix},{[testenv:pylint]maybe_someday} Lib/languages [flake8] @@ -65,7 +64,7 @@ exclude = # The ignore-list below comes from FontBakery linting setup. # The `languages` module has a much simples codebase, so we may be in much better shape in here. # So, perhaps, we could safely enable many of these lint-checks. -max-line-length=88 +max-line-length=100 ignore = # ITEMS BELOW ARE MEDIUM PRIORITY TO FIX AND REMOVE FROM IGNORE LIST E402, # module level import not at top of file