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

Type-check StatementParser #489

Merged
merged 2 commits into from May 11, 2017
Merged

Type-check StatementParser #489

merged 2 commits into from May 11, 2017

Conversation

ghost
Copy link

@ghost ghost commented Apr 25, 2017

Q A
Bug fix? no
Breaking change? no
New feature? no
Deprecations? no
Spec compliancy? no
Tests added/pass? Tests pass
Fixed tickets
License MIT

Enables type-checking in the file and annotates all methods.
Also switched ExportNamedDeclaration.specifiers to use a $ReadOnlyArray.

@codecov
Copy link

codecov bot commented Apr 25, 2017

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #489      +/-   ##
==========================================
+ Coverage   98.26%   98.27%   +<.01%     
==========================================
  Files          22       22              
  Lines        3526     3527       +1     
  Branches      978      978              
==========================================
+ Hits         3465     3466       +1     
  Misses         22       22              
  Partials       39       39
Flag Coverage Δ
#babel 81.08% <95.52%> (ø) ⬆️
#babylon 97.1% <100%> (ø) ⬆️
Impacted Files Coverage Δ
src/parser/statement.js 99.08% <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 1773ca7...05d5cf1. Read the comment docs.

@danez danez merged commit 7a8b64c into babel:master May 11, 2017
@ghost ghost deleted the statement branch May 11, 2017 14:35
@ghost ghost mentioned this pull request May 12, 2017
@ghost ghost mentioned this pull request May 19, 2017
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