This repository has been archived by the owner. It is now read-only.

Fix issues with default object params in async functions #96

Merged
merged 1 commit into from Aug 23, 2016

Conversation

Projects
None yet
3 participants
@danez
Copy link
Member

danez commented Aug 18, 2016

This fixes the other half of #92

const a = async ({bar = "bar"}) => {}

It was throwing an error because shorthands were not allowed in this scenario, by setting the correct params for parseExprListItem() when it is an async function fixes the problem.

Fix #92

@danez danez force-pushed the danez:dafault-params-92 branch from 704dc5b to 3d502a8 Aug 18, 2016

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Aug 18, 2016

Current coverage is 96.91% (diff: 100%)

No coverage report found for master at 4506822.

Powered by Codecov. Last update 4506822...3d502a8

@danez danez added the Tag: Bug Fix label Aug 19, 2016

Fix issues with default object params in async functions
This change allows async functions to have a spread argument which
defines a default value.

@danez danez force-pushed the danez:dafault-params-92 branch from 3d502a8 to 2c68435 Aug 23, 2016

@hzoo hzoo merged commit 2cfae60 into babel:master Aug 23, 2016

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@danez danez deleted the danez:dafault-params-92 branch Aug 31, 2016

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