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

chore(release): note breaking changes for v5 #347

Merged
merged 1 commit into from
Apr 18, 2024
Merged

chore(release): note breaking changes for v5 #347

merged 1 commit into from
Apr 18, 2024

Conversation

mcous
Copy link
Collaborator

@mcous mcous commented Apr 6, 2024

8d98aa5 landed in next with two issues that I didn't realize when I was writing my commit message:

  • semantic-release uses the angular commit spec rather than the conventional commit spec, which means it did not recognize the exclamation mark in fix!: ... as a breaking change
  • I typed BREAKING CHANGES: rather than BREAKING CHANGE: in the commit body

This means that the breaking changes in #325 were released improperly as v4.2.2 instead of v5.0.0 on the next dist tag. This PR adds an empty commit with the correct BREAKING CHANGE message below to hopefully trigger a major bump. I'm leaving this as a draft in case there are other breaking changes that should be incorporated in the v5 release.

BREAKING CHANGE: render is now more strict about the target option, and the container result is now the direct parent of your component instance.

BREAKING CHANGE: `render` is now more strict about the `target`
option, and the `container` result is now the direct parent of
your component instance.
@mcous
Copy link
Collaborator Author

mcous commented Apr 18, 2024

@yanick looks like we just had another release go out on next, did you see this PR and my notes?

@yanick
Copy link
Collaborator

yanick commented Apr 18, 2024

Yup, I am on this very page and was about to say "let's merge this, and get v5 out of the door".

So, uh, let's merge this, and get v5 out of the door. :-)

@yanick yanick marked this pull request as ready for review April 18, 2024 15:28
@yanick yanick merged commit 496c455 into testing-library:next Apr 18, 2024
15 of 16 checks passed
@mcous mcous deleted the bump-v5 branch April 18, 2024 15:29
Copy link

🎉 This PR is included in version 5.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 5.0.0 🎉

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants