-
Notifications
You must be signed in to change notification settings - Fork 622
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
CompactTarget
is missing LowerHex
and UpperHex
impls
#2219
Comments
Hi @Kixunil , I've changed only How to solve it? Should I just skip the hook? |
@crywolf yeah, just skip the hook. @tcharding I guess we should remove that from the githooks? Are they installed automatically somehow or do people need to manually remove the hooks now that we're not forcing the library to be formatted every day? |
I've skipped the hook. |
I deleted that rustfmt check from githook here #2222 so it should work for everybody now |
5a3f1a6 Remove nightly rustfmt check from pre-commit githook See #2135 (Vojtěch Toman) Pull request description: See #2135 and discussion [#2219#issuecomment-1826437013](#2219 (comment)) ACKs for top commit: Kixunil: ACK 5a3f1a6 apoelstra: ACK 5a3f1a6 Tree-SHA512: 3fabadcbc4f80fb133318138aad8e1baccbeffad57d09fe86ed12f1156e8c9505911f8fc3a785896832fad02e20a366e9d95d37602bdd5fc02d9c715a867cb72
These are useful and used in the wild.
The text was updated successfully, but these errors were encountered: