Skip to content
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 node 8 support #148

Merged
merged 5 commits into from
May 31, 2017
Merged

Add node 8 support #148

merged 5 commits into from
May 31, 2017

Conversation

stefanpenner
Copy link
Member

@stefanpenner stefanpenner commented May 31, 2017

resolves #147

@stefanpenner stefanpenner mentioned this pull request May 31, 2017
@stefanpenner stefanpenner force-pushed the node-8 branch 3 times, most recently from 319cc20 to 1c26ed2 Compare May 31, 2017 18:43
@stefanpenner stefanpenner requested a review from rwjblue May 31, 2017 19:54
@@ -691,7 +691,7 @@ describe('ember-cli-babel', function() {
expect(pluginRequired).to.be.true;
});

it('returns false when targets do not require plugin', function() {
it.only('returns false when targets do not require plugin', function() {
Copy link
Contributor

Choose a reason for hiding this comment

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

@stefanpenner sorry about that... was (foolishly) not even considering anyone else using isPluginRequired when we made string version target changes (originally targeted for 2.x) :(

Copy link
Member Author

Choose a reason for hiding this comment

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

@existentialism not a problem. Just a surprise, glad it was easy to work around.

@Turbo87 Turbo87 changed the title [BUGFIX #147] add node 8 support Add node 8 support Oct 27, 2017
siva-sundar pushed a commit to siva-sundar/ember-cli-babel that referenced this pull request Feb 11, 2021
Deserialize nested arrays in the parallel API correctly
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.

NodeJS 8.0.
3 participants