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

fix(organize-imports): Don't remove exported elements and react called functions #413

Merged
merged 3 commits into from
Mar 5, 2018

Conversation

buehler
Copy link
Owner

@buehler buehler commented Mar 5, 2018

  • Fixes #380
  • Fixes #364

Description

Updates the node-typescript parser to recognize exports as usages.

@buehler buehler changed the title Fix/reexported elements fix(organize-imports): Don't remove exported elements and react called functions Mar 5, 2018
@codecov
Copy link

codecov bot commented Mar 5, 2018

Codecov Report

Merging #413 into develop will increase coverage by 1.19%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #413      +/-   ##
===========================================
+ Coverage    72.26%   73.46%   +1.19%     
===========================================
  Files           29       29              
  Lines          941      942       +1     
  Branches       155      155              
===========================================
+ Hits           680      692      +12     
+ Misses         204      194      -10     
+ Partials        57       56       -1
Impacted Files Coverage Δ
src/imports/import-manager.ts 81.25% <ø> (+4.68%) ⬆️
src/imports/import-organizer.ts 53.12% <ø> (+12.5%) ⬆️
src/configuration/index.ts 88.46% <100%> (+0.46%) ⬆️
src/utilities/utility-functions.ts 66.37% <0%> (+0.88%) ⬆️

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 44fb21c...d9eab1e. Read the comment docs.

@buehler buehler merged commit 9348b40 into develop Mar 5, 2018
@buehler buehler deleted the fix/reexported-elements branch March 5, 2018 14:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant