Skip to content

Add Svg prefix to index.js exports by default#383

Merged
gregberge merged 2 commits into
gregberge:masterfrom
TrySound:svg-prefix
Jan 18, 2020
Merged

Add Svg prefix to index.js exports by default#383
gregberge merged 2 commits into
gregberge:masterfrom
TrySound:svg-prefix

Conversation

@TrySound

Copy link
Copy Markdown
Contributor

Ref #379

@vercel

vercel Bot commented Jan 17, 2020

Copy link
Copy Markdown

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/gregberge/svgr/ase50qs21
✅ Preview: https://svgr-git-fork-trysound-svg-prefix.gregberge.now.sh

@codecov

codecov Bot commented Jan 17, 2020

Copy link
Copy Markdown

Codecov Report

Merging #383 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #383   +/-   ##
=======================================
  Coverage   85.14%   85.14%           
=======================================
  Files          30       30           
  Lines         505      505           
  Branches      140      140           
=======================================
  Hits          430      430           
  Misses         62       62           
  Partials       13       13

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 f734dda...5d4627f. Read the comment docs.

@gregberge gregberge left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only if the name of the file starts with a number.

@gregberge gregberge merged commit fd120d1 into gregberge:master Jan 18, 2020
@TrySound TrySound deleted the svg-prefix branch January 18, 2020 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants