Skip to content

Conversation

@simonihmig
Copy link
Contributor

Fixes #1048

@richgt
Copy link
Contributor

richgt commented Jun 15, 2021

Appreciate the fix, but why did this not fail ember-try in its own tests (it seems to break ember-try everywhere else...)? Can we create a test that would have failed here so this doesn't break again later?

Copy link
Contributor

@richgt richgt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving since it seems to be blocking many people. Would love to see a test as a followup.

@rwjblue
Copy link
Member

rwjblue commented Jun 15, 2021

ZOMG, so sorry for this!!! I totally didn't look for an open PR when I opened #1050 🤦

I'm releasing that fix now

@rwjblue rwjblue closed this Jun 15, 2021
@rwjblue
Copy link
Member

rwjblue commented Jun 15, 2021

Approving since it seems to be blocking many people. Would love to see a test as a followup.

Ya, totally agree. I'm going to try to diagnose why this didn't fail our own CI.

@rwjblue
Copy link
Member

rwjblue commented Jun 15, 2021

Fix released as 2.2.8, sorry about this folks :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expected Broccoli node, got undefined for inputNodes[0]

3 participants