Skip to content

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
@mcous mcous deleted the bump-v5 branch April 18, 2024 15:29
@github-actions
Copy link

🎉 This PR is included in version 5.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
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