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
Add multiline workaround #744
Conversation
7965ab8
to
c519d5f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the quick fix and looks good if you could just review the comments and apply as necessary. Thanks!
Thank you @brettz9! Changes applied! I also tested it against the Gutenberg repo and it looked okay, although |
Thanks If you haven't run Also for next time, if you can run Thanks! |
Co-authored-by: Brett Zamir <brettz9@yahoo.com>
Co-authored-by: Brett Zamir <brettz9@yahoo.com>
e65a299
to
a880e33
Compare
🎉 This PR is included in version 34.8.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
It's a workaround for #743.
Basically, it ignores multiline tags. To check it, it'll verify in the tokens if
tag
is empty and it containstype
orname
. The ideal solution would be to align it even being multiline.