-
Notifications
You must be signed in to change notification settings - Fork 0
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
[LOOM-1377]: transform to mjs & replace svg2react with svgr #71
Conversation
Looks great! I guess the only thing missing is transpliation of tsx icons |
I was considering that this would be handled by bpk, as the dist get's Not ideal, however, if/when we wanted to remove that weirdness we can tackle transpilation at that point. Or do you think that is a bit dangerous? |
I'd rather go safe and not publish tsx to npm, in case someone is using this library directly (which can always be the case with OSS) |
Yup, that's fair. |
I think it looks good though is it worth updating/adding to the readme the process that happens to generate these svg components and how to manage their transformation with the svgr build file? Just for the next person to know how all this works. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love it!
Task
mjs
.jsx
not.js
.d.ts
typings files