Skip to content

style: Avoid optional parameter syntax#876

Merged
tmcw merged 3 commits intomasterfrom
no-optional-parameter
Aug 12, 2017
Merged

style: Avoid optional parameter syntax#876
tmcw merged 3 commits intomasterfrom
no-optional-parameter

Conversation

@tmcw
Copy link
Member

@tmcw tmcw commented Aug 11, 2017

Supporting this syntax uncorks a whole bottle of worms and requires participation in more of the
JavaScript ecosystem and is not worthwhile in my opinion at this time.

Fixes #873

Supporting this syntax uncorks a whole bottle of worms and requires participation in more of the
JavaScript ecosystem and is not worthwhile in my opinion at this time.

Fixes #873
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.08%) to 78.463% when pulling 30d12db on no-optional-parameter into ffdfa21 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.3%) to 78.255% when pulling c02b880 on no-optional-parameter into ffdfa21 on master.

@tmcw tmcw merged commit 25d0cfa into master Aug 12, 2017
@tmcw tmcw deleted the no-optional-parameter branch August 12, 2017 21:25
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.

2 participants