Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor pre-commit. #1278

Merged
merged 23 commits into from
Sep 6, 2024
Merged

Refactor pre-commit. #1278

merged 23 commits into from
Sep 6, 2024

Conversation

joaander
Copy link
Member

@joaander joaander commented Sep 4, 2024

Description

Update to the latest formatters and linters.

Motivation and Context

  • Newer versions of clang-tidy make many new recommendations to improve the code.
  • Ruff replaces both flake8 and black while being orders of magnitude faster to work with.

Checklist:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • I have updated the documentation (if relevant).
  • I have added tests that cover my changes (if relevant).
  • All new and existing tests passed.
  • I have updated the credits.
  • I have updated the Changelog.

@joaander joaander force-pushed the refactor-pre-commit branch 2 times, most recently from b11df95 to abdb303 Compare September 4, 2024 13:43
@joaander joaander force-pushed the refactor-pre-commit branch 4 times, most recently from 0ae6f97 to cb57280 Compare September 4, 2024 15:07
@joaander joaander force-pushed the refactor-pre-commit branch from cb57280 to bfe556f Compare September 4, 2024 15:19
@joaander joaander force-pushed the refactor-pre-commit branch from 1f1698a to 152a90f Compare September 4, 2024 18:39
@joaander joaander marked this pull request as ready for review September 4, 2024 18:56
@joaander joaander mentioned this pull request Sep 5, 2024
7 tasks
@joaander joaander requested a review from DomFijan September 5, 2024 13:14
Base automatically changed from refactor-build to nanobind September 6, 2024 13:09
@joaander joaander merged commit 4c0c338 into nanobind Sep 6, 2024
26 checks passed
@joaander joaander deleted the refactor-pre-commit branch September 6, 2024 13:09
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.

1 participant