-
Notifications
You must be signed in to change notification settings - Fork 31
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
Breaking Change included as a patch version #514
Comments
I thought it would check the content of each commit. |
No, we check the semver tag of the merge commit generate from the title of the PR and use that in the notify release workflow #488 |
🎉 This issue has been resolved in version 5.0.1 🎉 The release is available on: Your optic bot 📦🚀 |
Due to #502 and the engines increase (from 16 => 20), the latest patch versions of this package should have been considered a breaking change. This dropping of support for Node <= 20 doesn't appear to be documented either.
specifically - https://github.com/nearform/fast-jwt/pull/502/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L80
The text was updated successfully, but these errors were encountered: