Skip to content
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

fix: address broken internal git hooks #128

Merged
merged 2 commits into from
May 5, 2023
Merged

Conversation

Parsium
Copy link
Contributor

@Parsium Parsium commented Apr 28, 2023

Proposed behaviour

  • Update husky to version 8.x
  • Address husky breaking changes between versions 4.x to 8.x
  • Update all dependencies, including these to their latest versions:
    • @commitlint/cli from version 13.1.0 to 17.6.1
    • @commitlint/config-conventional from version 13.2.0 to 17.6.1
    • @semantic-release/changelog from version 5.0.1 to 6.0.3
    • @semantic-release/git from version 9.0.0 to 10.0.1

Current behaviour

  • Many packages are outdated and require upgrading.
  • Commit checks are not running after git commit - since we accidentally upgraded husky without addressing the introduced breaking changes (see PR chore(deps-dev): bump husky from 4.3.0 to 6.0.0 #55). This means its possible to write commit messages which semantic-release would not recognise.

Checklist

  • Release notes (Conventional Commits)
  • Unit tests
  • Readme updated

Testing instructions

To check post-commit actions are being triggered correctly, when making a commit, check the following actions are completed after git commit:

  • Prettier is run when javascript file is changed
  • Tests are run
  • Commitlint is run on commit message

Parsium added 2 commits April 28, 2023 16:29
Previously we upgraded husky without addressing the introduced
breaking changes, making it possible to write commit messages
which semantic-release would not recognise.
- Update husky to v8.x
- Address the breaking changes between v4.x and v8.x

see #55
Including updating these dependencies to their latest versions:
- @commitlint/cli to v17.6.1
- @commitlint/config-conventional to v17.6.1
- @semantic-release/changelog to v6.0.3
- @semantic-release/git to v10.0.1
@nineteen88
Copy link

Interesting that they've moved the config into actual config files now. Much better in my opinion

@Parsium Parsium merged commit d3255da into master May 5, 2023
@Parsium Parsium deleted the update-dependencies branch May 5, 2023 09:16
@carbonci
Copy link
Collaborator

carbonci commented May 5, 2023

🎉 This PR is included in version 1.14.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants