Skip to content

Commit

Permalink
Old pylint version not compatible with python 3.11
Browse files Browse the repository at this point in the history
Deprecated github action to use 3.10 instead of 3.11, and then updated
code_consistency.py to account for new pylint warning about formatting
u-strings
  • Loading branch information
mnlevy1981 committed Feb 14, 2023
1 parent 3c1b3b3 commit 1d057e0
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/run_test_suite.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ jobs:
strategy:
matrix:
python-version:
- 3.11
- '3.10'

steps:
- uses: actions/checkout@v3
Expand Down
8 changes: 4 additions & 4 deletions MARBL_tools/code_consistency.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
import logging
from collections import deque # Faster pop / append than standard lists
from collections import OrderedDict
UNIBLOCK = u"\u2588"
UNIBLOCK = u"\u2588" # pylint: disable=redundant-u-string-prefix

##############

Expand Down Expand Up @@ -218,8 +218,8 @@ def check_r8_settings(self, file_and_line, line, comment_char="!"):
else:
MARBL_ROOT = os.path.join(SCRIPT_DIR, '..')

fortran_files = [] # pylint: disable=C0103
python_files = [] # pylint: disable=C0103
fortran_files = []
python_files = []
for root, dirs, files in os.walk(MARBL_ROOT):
for thisfile in files:
if thisfile.endswith(".F90"):
Expand All @@ -231,7 +231,7 @@ def check_r8_settings(self, file_and_line, line, comment_char="!"):
logging.basicConfig(format='%(message)s', level=logging.DEBUG)
LOGGER = logging.getLogger(__name__)

Tests = ConsistencyTestClass() # pylint: disable=C0103
Tests = ConsistencyTestClass()

# Fortran error checks
LOGGER.info("Check Fortran files for coding standard violations:")
Expand Down

0 comments on commit 1d057e0

Please sign in to comment.