Add ruff format linter and fix current codebase#120
Add ruff format linter and fix current codebase#120scal444 merged 2 commits intoNVIDIA-Digital-Bio:mainfrom
Conversation
|
| Filename | Overview |
|---|---|
| .github/workflows/lint.yml | Adds new ruff-format CI job that installs ruff 0.15.8 and runs the format check in dry-run mode; straightforward and consistent with the existing lint job pattern. |
| admin/run_ruff_format.sh | New shell script that validates the installed ruff version, accepts a -d flag for dry-run (--check) mode, and runs ruff format over the repo root; well-structured and consistent with sibling scripts. |
| pyproject.toml | Expands ruff's include list from just nvmolkit/tests/conftest.py to all Python files in nvmolkit, benchmarks, and setup.py, aligning the linter scope with the files actually reformatted in this PR. |
| nvmolkit/similarity.py | Pure formatting changes: function signatures reformatted to one parameter per line, long ternary expressions wrapped in parentheses, blank lines added between top-level definitions; no logic changes. |
| nvmolkit/tests/test_substructure.py | Large formatting-only pass: blank lines added before class definitions, trailing whitespace removed from docstrings, multi-line assert messages reformatted, alignment-style spaces removed from list literals. |
Reviews (2): Last reviewed commit: "Ruff version pin and chekc" | Re-trigger Greptile
evasnow1992
left a comment
There was a problem hiding this comment.
Changes look good to me. Thank you!
No description provided.