Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Apply some Pylint fixes : Part 1 #1591

Merged
merged 15 commits into from
Nov 3, 2023
5 changes: 4 additions & 1 deletion .github/workflows/pylint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,10 @@ jobs:
id: pylint
run: |
echo $GITHUB_WORKFLOW_SHA
python -m pylint --rcfile=.pylintrc pyccel/parser/semantic.py | tee pylint_output.out || true
ast_files="bind_c.py bitwise_operators.py builtin_imports.py c_concepts.py class_defs.py cmathext.py itertoolsext.py omp.py operators.py scipyext.py sympy_helper.py sysext.py type_annotations.py"
pylintable_files="pyccel/parser/semantic.py"
for f in $ast_files; do pylintable_files="$pylintable_files pyccel/ast/$f"; done
python -m pylint --rcfile=.pylintrc $pylintable_files | tee pylint_output.out || true
python ci_tools/parse_pylint_output.py pylint_output.out $GITHUB_STEP_SUMMARY
shell: bash
- name: "Post completed"
Expand Down
2 changes: 1 addition & 1 deletion ci_tools/coverage_analysis_tools.py
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ def allow_untested_debug_code(untested):
for l in line_nums:
line = f_lines[l-1]
n = len(line)-len(line.lstrip())
i = l
i = l-1
func_found = ''
while i >= 0 and not func_found:
line = f_lines[i]
Expand Down
Loading