-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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(glimmer): {{else}}{{#if}} into {{else if}} merging #6080
Conversation
49bf6bd
to
9a3ee3d
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.
awesome!! 👍
9a3ee3d
to
6df3a03
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!
Need add note in changelog
@evilebottnawi done, sorry for that |
ea48afb
to
26e6d8e
Compare
``` {{#if a}} a {{else}} {{#if c}} c {{/if}} e {{/if}} ``` should be left untouched. Instead, it is turned into: ``` {{#if a}} a {{else if c}} c e {{/if}} ```
@evilebottnawi |
@dcyriller Thanks, can you fix CI? |
@evilebottnawi CI is now 🍏 I see that you squash the commits, so I don't bother to remove the last one (that was empty). |
/cc @prettier/core we need rule about merging (minimum 2 approve from core as example) |
Description
The first commit highlights the issue. The second fixes it.
The following code should be left untouched:
Instead, it was turned into:
Checks
CHANGELOG.unreleased.md
file following the template.Playground
✨Try the playground for this PR✨