Skip to content

Subsequent Init runs would clear Husky pre-commit #1520#1521

Open
fgrande wants to merge 1 commit intotypicode:mainfrom
fgrande:fg/1520
Open

Subsequent Init runs would clear Husky pre-commit #1520#1521
fgrande wants to merge 1 commit intotypicode:mainfrom
fgrande:fg/1520

Conversation

@fgrande
Copy link

@fgrande fgrande commented Sep 16, 2024

Hi,
as stated in #1520 I'm sending You a PR.

In my case, we share the same project between many coleagues. If one would run - erroneously - the "init", would delete the pre-commit hook. This is why - IMHO - would be fine to preserve it.

Regards
FabioG

@Munaf-Divan
Copy link

Munaf-Divan commented Oct 7, 2024

This is a good use case and addresses my needs.

Out of curiosity, would this require any changes in tests files, @fgrande?

@fgrande
Copy link
Author

fgrande commented Oct 7, 2024

@Munaf-Divan I haven't added a new test for this in my PR. Maybe it can be nice to have (let me see if I have time to implement it)

@AspadaX
Copy link

AspadaX commented Jan 21, 2025

Can anyone please merge this PR. This feature is really helpful. The current version almost renders our team project's pre-commit unusable.

@ben-buitendijk
Copy link

I would also like to see this merged!

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.

4 participants