Add official support for pre-commit.com (python tool)

[fregante]

See https://pre-commit.com

I think it's a matter of specifying a .pre-commit-hooks.yaml file in the root of the repo, then it becomes available as:

  - repo: https://github.com/keithamus/sort-package-json
    rev: v3.0.0
    hooks:
      - id: sort-package-json

Links:

• Existing hooks: https://pre-commit.com/hooks.html
• Guide: https://pre-commit.com/index.html#new-hooks
• Biome's setup for example: https://github.com/biomejs/pre-commit/blob/main/.pre-commit-hooks.yaml

[fisker]

Maintenance experience for pre-commit for Prettier is nightmare to me.

[fisker]

In short

1. Prettier add a seperate repo for better support
2. pre-commit started a new repo, don't know why, no communication. I was glad someone take over, since I don't use it myself, so I achieved repo in Prettier immediately 😄
3. After Prettier starts release v4, there are some issue raised, pre-commit just abandon their own repo (I don't see a reasonable explanation)

Story starts from prettier/prettier#8925

[fisker]

Anyway, I won't block this, just want share the story.

[fregante]

Yep, I'm pretty familiar with prettier/pre-commit story, I tried to fix it but the maintainer blocked my PR and deleted all issues. Unfortunately the tool itself is still very good so I'm stuck with it.

This wouldn't be affected because the problem was specifically due to the lack of first-party support (and bad version mirroring). If you were to add a file to this repo, I reckon it would be straightforward and trouble free.

[fregante]

Story starts from prettier/prettier#8925

Love me some GitHub drama, will read up.

[fregante]

While we're here, why not exhume https://github.com/prettier/pre-commit? Seems like a saner option to the broken mirror creator

[fisker]

I don't think people use the file in eslint repo https://github.com/eslint/eslint/blob/30ae4ed093d19e9950d09c2ab57f43d3564e31c9/.pre-commit-hooks.yaml#L1

Since there is a eslint-mirror. The owner of eslint doesn't even know what it for eslint/eslint#18643 (comment)

[fisker]

While we're here, why not exhume https://github.com/prettier/pre-commit? Seems like a saner option to the broken mirror creator

Do you want maintain?

[fregante]

I'm good. 😅

Anyway, since sort-package-json is much lighter/simpler than prettier, I think it's as easy as adding a file and forgetting about it, right? Optimizations like prettier/prettier#8925 would be ignored.

[fisker]

I'll let keithamus decide.

[keithamus]

Seems fine. If we add it can we somehow add some way to exercise the tool within CI? sort-package-json has a wide array of contributors and I'd hate it if we accidentally broke some integration because no one was keeping an eye on it.