Add pre-commit hooks for Python & Scala, configure Black/Ruff/Flake8/scalafmt, and reformat code #65
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…scalafmt, and reformat code - Introduce .pre-commit-config.yaml with: • Black, Ruff, and Flake8 for Python (isolation-forest-onnx) • Local scalafmt hook for Scala (isolation-forest) - Add isolation-forest-onnx/pyproject.toml to configure Black (line-length=88, py39 target, excludes) - Update setup.cfg in isolation-forest-onnx to align Flake8 max-line-length=88 - Apply Black/Flake8 fixes to Python code (notable quoting, line breaks, docstring formatting) - Add isolation-forest/scalafmt.conf for consistent Scala style (maxColumn=100, trailingCommas=always, etc.) - Reformat Scala code in isolation-forest (extensive line-wrapping, docstring updates, alignment) - Minor cleanups in test files (removing redundant prints, adjusting lines) - Ensures consistent code style and prepares repository for automated pre-commit checks
|
这是来自QQ邮箱的假期自动回复邮件。王中情：您好，已收到您的邮件，我会尽快给您回复。谢谢您的邮件！祝您生活愉快！
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add pre-commit hooks for Python & Scala, configure Black/Ruff/Flake8/scalafmt, and reformat code
Overview
This PR introduces a consistent style/linting workflow for both Python and Scala modules in this project. It adds a new
.pre-commit-config.yamlplus supporting configuration files so that any staged changes will be automatically formatted and checked before commit.Key Highlights
.pre-commit-config.yaml
isolation-forest-onnx/isolation-forest/Python
pyproject.tomlinisolation-forest-onnx/for Black (88-char line length,py39target).setup.cfgfor Flake8 to match line length = 88 and ignore certain dirs.Scala
scalafmt.confinisolation-forest/(maxColumn=100, trailing commas, docstring style, etc.).General Cleanup
How to Use
Install/Update pre-commit locally:
Install scalafmt (for Scala formatting), for example:
Re-install hooks (at the repository root):
Verify by committing any changes to
.pyor.scalafiles. The configured hooks will run automatically, formatting code and checking for lint errors.Testing
pre-commit run --all-filesto confirm hooks apply cleanly on the entire codebase.pytest) and Scala (e.g.,sbt testor Gradle equivalent).