Include src in npm (for types)

[IanVS]

Summary

I tried to use version 6.0.0-alpha.0 to get the typescript definitions as added in #555, but they are in core/src, which is not currently included in the npm package. This change includes src, so that the types can be used by consuming packages.

This also fixes some types that were commented on in the PR but not addressed.

Test plan

I copied the /src folder into node_modules/@svgr/core/ and verified that types started working correctly in my consuming project.

[vercel]

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

🔍 Inspect: https://vercel.com/gregberge/svgr/2tTnbvGULQpMw4VJrfWgWJya4FxG
✅ Preview: https://svgr-git-fork-ianvs-expose-types-gregberge.vercel.app

[gregberge]

Fixed in b7db87f

[IanVS]

Just curious, what's the reason for committing directly instead of merging my PR? I would have liked to be considered a contributor to the project…

[gregberge]

Just curious, what's the reason for committing directly instead of merging my PR? I would have liked to be considered a contributor to the project…

In your PR you includes everything under src, but we only need one file in lib. Your PR is wrong, I wanted to go faster so I made the fix directly. Anyway, thanks for your work 🤗

[SimenB]

@gregberge can you publish as well?

[gregberge]

@SimenB I have a few work ongoing on v6, please be patient :)