Skip to content

Conversation

@enrico-lattuada
Copy link
Collaborator

@enrico-lattuada enrico-lattuada commented Jun 5, 2025

Description

  • Create a pre-commit-config.yaml running linters, formatters, sorter, static checkers
  • Install hooks and verify pre-commit run --all-files passes on the migrated code
  • Update install.py
  • Update dev dependencies

Motivation and context

Pre-commit configs enhance code quality, reduce development time, and improve collaboration. These hooks automate tasks like linting, formatting, and testing before a commit, ensuring code adheres to standards and reduces errors.

Resolves #248

How has this been tested?

  • CI pipelines
  • Locally by running tests after installation with:
    • Arch Linux (Linux 6.14.9), in virtual environment, using gcc 15.1.1, cuda 12.8, with cuda enabled
    • Windows 11, in both global and in virtual environment (conda), using visual studio compiler, cuda 12.9, with cuda enabled

Change log

v0.3.14 (2025-MM-DD)
^^^^^^^^^^^^^^^^^^^^

*Added*

- Integrated [pre-commit](https://pre-commit.com/) hooks to enforce code quality checks automatically before commits.

Checklist:

@enrico-lattuada enrico-lattuada self-assigned this Jun 5, 2025
@enrico-lattuada enrico-lattuada added priority: high Important improvements or optimizations type: performance Speedups, GPU‐acceleration, or memory optimization difficulty: medium Standard tasks that need moderate effort labels Jun 5, 2025
@enrico-lattuada enrico-lattuada moved this to In progress in Modernization Roadmap Jun 5, 2025
@enrico-lattuada enrico-lattuada linked an issue Jun 5, 2025 that may be closed by this pull request
2 tasks
@enrico-lattuada enrico-lattuada changed the base branch from 252-devops-finalize-phase-1 to develop December 7, 2025 19:04
@enrico-lattuada enrico-lattuada marked this pull request as ready for review December 7, 2025 19:16
@enrico-lattuada
Copy link
Collaborator Author

Hi @asortino! If you have some time, would you be willing to review this pull request? I'd really appreciate your feedback. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

difficulty: medium Standard tasks that need moderate effort priority: high Important improvements or optimizations type: performance Speedups, GPU‐acceleration, or memory optimization

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

[DevOps] Pre-commit configuration

2 participants