Skip to content
This repository has been archived by the owner on May 19, 2018. It is now read-only.

Don't add 'await' property to ForInStatement #504

Merged
merged 1 commit into from May 2, 2017
Merged

Don't add 'await' property to ForInStatement #504

merged 1 commit into from May 2, 2017

Conversation

ghost
Copy link

@ghost ghost commented May 2, 2017

Q A
Bug fix? yes
Breaking change? yes (minor)
New feature? no
Deprecations? no
Spec compliancy? yes
Tests added/pass? Tests modified to pass
Fixed tickets
License MIT

Assigns await only to ForOfStatement nodes as described in the spec.

@codecov
Copy link

codecov bot commented May 2, 2017

Codecov Report

Merging #504 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #504      +/-   ##
==========================================
+ Coverage   98.26%   98.26%   +<.01%     
==========================================
  Files          22       22              
  Lines        3517     3518       +1     
  Branches      973      974       +1     
==========================================
+ Hits         3456     3457       +1     
  Misses         22       22              
  Partials       39       39
Flag Coverage Δ
#babel 80.95% <100%> (ø) ⬆️
#babylon 97.1% <100%> (ø) ⬆️
Impacted Files Coverage Δ
src/parser/statement.js 99.07% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e81b5f8...553b065. Read the comment docs.

@hzoo hzoo merged commit 3a6b77f into babel:master May 2, 2017
@ghost ghost deleted the for-await-in branch May 2, 2017 19:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants