[allura] 01/02: [#8502] updated ruff config and added an extra flag --show-source

[gc...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

gcruz pushed a commit to branch gc/8502
in repository https://gitbox.apache.org/repos/asf/allura.git

commit 42332fd0fe57d96778f45d38e534699cb50fb371
Author: Guillermo Cruz <gu...@slashdotmedia.com>
AuthorDate: Thu Mar 9 21:19:48 2023 +0000

    [#8502] updated ruff config and added an extra flag --show-source
---
 AlluraTest/alluratest/test_syntax.py | 2 +-
 requirements.in                      | 3 +--
 ruff.toml                            | 2 +-
 3 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/AlluraTest/alluratest/test_syntax.py b/AlluraTest/alluratest/test_syntax.py
index 82bca77e5..7272da81f 100644
--- a/AlluraTest/alluratest/test_syntax.py
+++ b/AlluraTest/alluratest/test_syntax.py
@@ -80,7 +80,7 @@ def run_linter(files):
 def run_pyflakes(files):
     # skip some that aren't critical errors
     files = [f for f in files if '/migrations/' not in f]
-    cmd = "ruff check " + ' '.join(files) + " | grep -v '" + "'"
+    cmd = "ruff check " + ' '.join(files) + " --show-source"
     if run(cmd) != 1:
         # print 'Command was: %s' % cmd
         raise Exception('ruff failure, see stdout')
diff --git a/requirements.in b/requirements.in
index 68685bc65..e209b3487 100644
--- a/requirements.in
+++ b/requirements.in
@@ -49,7 +49,7 @@ werkzeug
 wrapt
 # testing
 mock
-#pyflakes
+ruff
 #pylint -- disabled due to [#8346]  (also requires diff versions on py2 vs 3, including transitive deps which gets tricky with pip-compile)
 testfixtures
 WebTest
@@ -59,4 +59,3 @@ pytest-sugar
 
 # deployment
 gunicorn
-ruff
\ No newline at end of file
diff --git a/ruff.toml b/ruff.toml
index 452bc05b8..7ea3f6a70 100644
--- a/ruff.toml
+++ b/ruff.toml
@@ -3,7 +3,7 @@ exclude = [
     'node_modules',
 ]
 per-file-ignores = {'__init__.py' = ['F403']}
-select = ['E', 'F', 'RUF100']
+select = ['E', 'F', 'RUF100', 'W6', 'ISC001']
 ignore = [
     'F401', # Imported but unused,
     'F811', # Redefinition of unused