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

Improve error message on attempt to destructure named import #288

Merged
merged 1 commit into from Jan 14, 2017

Conversation

Projects
None yet
4 participants
@existentialism
Copy link
Member

existentialism commented Jan 12, 2017

Q A
Bug fix? N
Breaking change? N
New feature? Y
Deprecations? N
Spec compliancy? N
Tests added/pass? Y
Fixed tickets Fixes #258
License MIT
@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Jan 12, 2017

Current coverage is 97.47% (diff: 100%)

Merging #288 into master will increase coverage by <.01%

@@             master       #288   diff @@
==========================================
  Files            21         21          
  Lines          3991       3993     +2   
  Methods         488        488          
  Messages          0          0          
  Branches       1167       1168     +1   
==========================================
+ Hits           3890       3892     +2   
  Misses           44         44          
  Partials         57         57          

Powered by Codecov. Last update aa1a4be...f231993

@danez

danez approved these changes Jan 12, 2017

@danez danez added the Tag: Polish label Jan 14, 2017

@danez danez merged commit 9d78213 into master Jan 14, 2017

4 checks passed

codecov/patch 100% of diff hit (target 97.46%)
Details
codecov/project 97.47% (+<.01%) compared to aa1a4be
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@danez danez deleted the issue258 branch Jan 14, 2017

@hzoo

This comment has been minimized.

Copy link
Member

hzoo commented Jan 14, 2017

Awesome stuff! Forgot I created the issue lol

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