You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mahout.apache.org by "guan404ming (via GitHub)" <gi...@apache.org> on 2025/11/12 10:17:29 UTC
[I] [Feature] Integrate more pre-commit hooks to improve code quality [mahout]
guan404ming opened a new issue, #618:
URL: https://github.com/apache/mahout/issues/618
### Summary
Integrate additional pre-commit hooks to improve code quality and maintain consistency across the Qumat codebase.
1. Ruff (linting & formatting) - already in dev dependencies
2. Python-specific checks - docstring placement, debug statement detection
3. Jupyter notebook cleanup - strip outputs before commit
4. Security scanning - Bandit for vulnerability detection
5. File validation - check large files, merge conflicts, TOML/JSON syntax
### Use Case
Currently, the project has basic pre-commit hooks.This reduces code review burden and catches common issues in CI/CD.
### Proposed Implementation
Configuration would be added incrementally to avoid disrupting existing workflows.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@mahout.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Feature] Integrate more pre-commit hooks to improve code quality [mahout]
Posted by "guan404ming (via GitHub)" <gi...@apache.org>.
guan404ming closed issue #618: [Feature] Integrate more pre-commit hooks to improve code quality
URL: https://github.com/apache/mahout/issues/618
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@mahout.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Feature] Integrate more pre-commit hooks to improve code quality [mahout]
Posted by "guan404ming (via GitHub)" <gi...@apache.org>.
guan404ming commented on issue #618:
URL: https://github.com/apache/mahout/issues/618#issuecomment-3582239873
I think for current usage these two hooks is enough. Feel free to reopen this, thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@mahout.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org