Skip to content

Add pre-commit#853

Merged
alancleary merged 11 commits into
mainfrom
alancleary/VCF-55/pre-commit
Nov 4, 2025
Merged

Add pre-commit#853
alancleary merged 11 commits into
mainfrom
alancleary/VCF-55/pre-commit

Conversation

@alancleary

Copy link
Copy Markdown
Member

This PR adds pre-commit to manage and automate code formatting.

The configuration runs the formatters currently used by the project, i.e. clang-format v15 and black v25.
These will now be handled automatically with pre-commit and GitHub workflows.
This module was already deprecated and is being removed since ClangFormat is now managed by pre-commit.
This is because formatting is now handled via the pre-commit CI. Note that pylint was removed here as well since this is the only place in the project that it is used.
This has been replaced by pre-commit.
This is because black is now managed by pre-commit.
Specifically, clang-format was dropped from the Java API documentation and pre-commit in now mentioned in the contributing documentation.
@alancleary alancleary added the enhancement New feature or request label Oct 31, 2025
@alancleary
alancleary marked this pull request as ready for review October 31, 2025 21:53
@alancleary
alancleary requested a review from jp-dark October 31, 2025 21:53
@alancleary
alancleary merged commit 54f192f into main Nov 4, 2025
11 checks passed
@alancleary
alancleary deleted the alancleary/VCF-55/pre-commit branch November 4, 2025 14:25
alancleary added a commit that referenced this pull request Nov 12, 2025
* Add a new Assets directory to examples (#849)

* Add pre-commit (#853)

* Consolidate URI functions (#851)

* Update TileDB core to 2.29.2 (#858)

---------

Co-authored-by: Julia Dark <24235303+jp-dark@users.noreply.github.com>
Co-authored-by: Nick Vigilante <nickvigilante@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants