-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Rich text: rename index.js > index.ts for type-only exports #50212
Conversation
- renames rich-text/src/index.js to rich-text/src/index.ts so that it can export the type as a type
Size Change: -12 B (0%) Total Size: 1.37 MB
ℹ️ View Unchanged
|
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.
Looks good, works well, and makes sense, I only have a minor correction suggestion.
Thanks @ramonjd 🙌 🚀
Co-authored-by: Marin Atanasov <[email protected]>
Thanks so much for this! I must not have caught it because the warnings didn’t cause an error in CI, and I was relying on tsc for local verification :) |
What, why and how?
When compiling the project, Webpack was whining at me:
"Enough!" I said, and I muted it by renaming index.js > index.ts and exporting
RichTextValue
as a type.Related reading:
I don't know if this is the best way to go about it. 🤷 Checking with @noahtallen from #49651
Testing Instructions
Run
npm run dev
You shouldn't see the following error in your console output.