Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Updating @svgr/core to 6.2.1 enabling the output of an index.tsx + adding react-native example #111

Merged

Conversation

MateusAndrade
Copy link
Contributor

@MateusAndrade MateusAndrade commented Jan 30, 2022

Hi! 👋 Before anything, thank you so much for the work on this cli. 🚀

Following that, I added a small update to @svgr/core updating the version to 6.2.1, outputting the index file as a tsx instead of an index.js.

Also, I added small example support react-native with Typescript along a transform removing invalid properties that aren;t supported by react-native-svg

We can look at the changelog here: gregberge/svgr@v6.1.2...v6.2.1

…o output and index.tsx instead of an index.js file
…dding TS config + removing XMLNS from the transform
Copy link
Owner

@marcomontalbano marcomontalbano left a comment

Choose a reason for hiding this comment

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

Hi @MateusAndrade, very glad to read your appreciation and thanks a lot for this contribution!
I would only add a description with the list of used svgr plugins and then I'll merge it into 4.1.0.

.figmaexportrc-native.example.ts Show resolved Hide resolved
Co-authored-by: Marco Montalbano <marcomontalbano.work@gmail.com>
@MateusAndrade
Copy link
Contributor Author

Btw, it looks like the coverage step broke, but I couldn't find specifically why: https://github.com/marcomontalbano/figma-export/runs/4998391478?check_suite_focus=true.

@marcomontalbano
Copy link
Owner

No problem for the coverage failure. Sometimes happen for PRs.
I just read this coverallsapp/github-action#26 and I'll try to switch to checkout@v2 🤞

@marcomontalbano marcomontalbano merged commit ea34d7f into marcomontalbano:master Jan 30, 2022
@MateusAndrade MateusAndrade deleted the feat/update-svgr-core branch January 30, 2022 19:33
@marcomontalbano marcomontalbano added the PR: New Feature 🚀 Only for pull request. New feature label Jan 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: New Feature 🚀 Only for pull request. New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants