Skip to content

SRE-3706: Add Python linting with flake8 + pylint#6

Merged
JohnMalmberg merged 1 commit into
mainfrom
sre-3706-python
May 19, 2026
Merged

SRE-3706: Add Python linting with flake8 + pylint#6
JohnMalmberg merged 1 commit into
mainfrom
sre-3706-python

Conversation

@JohnMalmberg

Copy link
Copy Markdown
Contributor
  • Add changed-file detection and runner wiring for Python files.
  • Run flake8 and pylint as a unified meta-linter on discovered *.py files and files without an extension that declare a Python shebang.

Changed files/directories

  • bin/run-linters.sh, checks/detect-linters.sh, checks/linter-common.sh: add linter_is_python_candidate() with shebang detection for files without an extension, python_needed flag, and dispatch wiring.
  • checks/install-linter-tools.sh: add flake8 and pylint to package list with dnf/yum name mapping.
  • checks/linters/python/: add Python meta-linter scripts (run.sh and run.ps1) running both flake8 and pylint on the same file set.
  • docs/linters.md: update linter list with python description and tool requirements.
  • vscode-project-words.txt: add flake8, pylint to project word list.

@JohnMalmberg JohnMalmberg requested a review from a team as a code owner April 24, 2026 18:58
@JohnMalmberg

Copy link
Copy Markdown
Contributor Author

https://github.com/daos-stack/system-pipeline-lib/actions/runs/24907131797/job/72939260503?pr=21 is test of this as a submodule with the expected result.

@JohnMalmberg JohnMalmberg force-pushed the sre-3706-groovylint branch 2 times, most recently from 84a3f9a to 9761338 Compare April 27, 2026 16:48
Base automatically changed from sre-3706-groovylint to main April 27, 2026 16:52
- Add changed-file detection and runner wiring for Python files.
- Run flake8 and pylint as a unified meta-linter on discovered *.py
  files and files without an extension that declare a Python shebang.

Changed files/directories
- bin/run-linters.sh, checks/detect-linters.sh, checks/linter-common.sh:
	add linter_is_python_candidate() with shebang detection for files
	without an extension, python_needed flag, and dispatch wiring.
- checks/install-linter-tools.sh:
	add flake8 and pylint to package list with dnf/yum name mapping.
- checks/linters/python/:
	add Python meta-linter scripts (run.sh and run.ps1) running both
	flake8 and pylint on the same file set.
- docs/linters.md:
	update linter list with python description and tool requirements.
- vscode-project-words.txt:
	add flake8, pylint to project word list.

Signed-off-by: John E. Malmberg <john.malmberg@hpe.com>
Comment thread checks/linters/python/run.sh
Comment thread checks/linter-common.sh
Comment thread checks/linter-common.sh
@JohnMalmberg JohnMalmberg merged commit ad19ea9 into main May 19, 2026
5 checks passed
@JohnMalmberg JohnMalmberg deleted the sre-3706-python branch May 19, 2026 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants