prettier / pre-commit

Mirror of Prettier package for pre-commit.
5 stars 6 forks source link

All tags are gone!? #26

Open WhyNotHugo opened 4 years ago

WhyNotHugo commented 4 years ago

All the release tags are gone.

This completely brakes existing integrations. It also breaks pre-commit autoupdate. Since there are no tags, the latest commit is picked up as the latest version:

-  - repo: https://github.com/prettier/prettier
-    rev: v2.1.2
+  - repo: https://github.com/prettier/pre-commit
+    rev: 413fd9da8e03249b74acf8543567e5debbad8bfa

I moved to using the pre-commit hook recently since prettier/prettier broke due to npm changes, but it seems this mirror is not broken too.

graingert commented 4 years ago

but it seems this mirror is not broken too.

it seems not not broken to me

graingert commented 4 years ago

this is also broken on pre-commit.ci where the environment is cached globally based on the revision, and so pushing to a branch won't update the code

graingert commented 4 years ago

@WhyNotHugo this is now fixed in https://github.com/pre-commit/mirrors-prettier

WhyNotHugo commented 4 years ago

Thanks, @graingert!