-
Notifications
You must be signed in to change notification settings - Fork 38
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(print): fix print for NullLiteral #725
fix(print): fix print for NullLiteral #725
Conversation
960fc6f
to
be1b4c3
Compare
@dcyriller may be you can help me 馃懠 |
This reverts commit be1b4c3.
It's never passing into my code. My test is actually wrong 馃槥 |
I finally found a way to reproduce the issue. It's when you are deleting or setting something on the node |
I can't merge. I let you do it 馃懠 |
@bmish can you merge it and create a patch ? 馃檹 |
@dcyriller want to release it? If you can't get to it, I might be able to try later today. |
It is released in v6.1.3 |
@bmish I tried with this fix it seems the test for PR ember-template-lint/ember-template-lint#2271 on
{{contact-null null}}
pass. I don't know how it was working before 馃