change module name to index and remove npmignore #1956
Conversation
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 41205fd:
|
|
Size Change: +9.12 kB (11%) Total Size: 78.6 kB
|
| "main": "dist/index.js", | ||
| "main:ie": "dist/index.ie11.js", | ||
| "umd:main": "dist/index.umd.js", | ||
| "unpkg": "dist/index.umd.min.js", | ||
| "module": "dist/index.esm.js", | ||
| "module:min": "dist/index.min.esm.js", |
There was a problem hiding this comment.
Are main:ie and umd:main and so on used in only rollup config?
There was a problem hiding this comment.
yes, is there any issues?
There was a problem hiding this comment.
There are no issues. I just wanted to hear the reason :)
There was a problem hiding this comment.
main:ie is for IE users, umd for users who use umd.
There was a problem hiding this comment.
Exactly, I wanted to confirm while these variables are used only in the config or not.
|
I noticed that https://react-hook-form.com/faqs#BrowserSupport still points to the old path. Should this be |
|
send a PR please @ryansully |
No description provided.