Skip to content

Husky no longer working after update back in April #2292

Description

@trusktr

Back in April we updated Husky from v3 to v8 due to Dependabot.

This broke our workflow: the git commit hook is no longer installed on a fresh clone with npm install, so we no longer run lint on commit.

Latest Husky requires manually installing hooks with npx husky install.

They recommend that the prepare script can be used to automate this for npm install. But then it runs at random other times like any time the package is being packed, f.e. during publish.

We can, however, make npx husky install be optional: the contributing guide can recommend it as a convenience for pre-pull-request time saving. I like making it opt-in, and those who don't want it can rely on pull request builds or running command manually. Wdyt?

Activity

  1. trusktr commented on Oct 22, 2023

    @trusktr
    MemberAuthor
  2. Koooooo-7 commented on Oct 22, 2023

    @Koooooo-7
    Member

    Yea, I have to say the github actions is the final gateway for lint check for now.
    Update the contributing guide on this is for sure.
    And I will try to find a more convenient way in further instead of contributors notice the lint issue after creating PRs.

    or, how about downgrade to the older version since we don't need much feats on husky now.

  3. Koooooo-7 commented on Oct 22, 2023

    @Koooooo-7
    Member
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions