Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Allow empty export statements #338

Merged
merged 1 commit into from
Nov 18, 2018
Merged

Conversation

alangpierce
Copy link
Owner

Fixes #335

@codecov-io
Copy link

Codecov Report

Merging #338 into master will increase coverage by 0.2%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #338     +/-   ##
=========================================
+ Coverage   77.05%   77.26%   +0.2%     
=========================================
  Files          41       41             
  Lines        5479     5305    -174     
  Branches     1286     1288      +2     
=========================================
- Hits         4222     4099    -123     
+ Misses        886      836     -50     
+ Partials      371      370      -1
Impacted Files Coverage Δ
src/CJSImportProcessor.ts 90.43% <100%> (+0.13%) ⬆️
src/transformers/CJSImportTransformer.ts 91.55% <100%> (+0.08%) ⬆️
src/util/getNonTypeIdentifiers.ts 75% <0%> (-25%) ⬇️
src/parser/traverser/util.ts 70.58% <0%> (-13.29%) ⬇️
src/parser/index.ts 75% <0%> (-9.62%) ⬇️
src/parser/util/whitespace.ts 80% <0%> (-7.5%) ⬇️
src/parser/plugins/types.ts 81.25% <0%> (-6.25%) ⬇️
src/computeSourceMap.ts 83.33% <0%> (-5.56%) ⬇️
src/index.ts 84.84% <0%> (-3.79%) ⬇️
src/identifyShadowedGlobals.ts 86.11% <0%> (-1.39%) ⬇️
... and 13 more

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 9fdabab...8e01184. Read the comment docs.

@alangpierce alangpierce merged commit de4c909 into master Nov 18, 2018
@alangpierce alangpierce deleted the allow-empty-export-statements branch November 18, 2018 16:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants