-
-
Notifications
You must be signed in to change notification settings - Fork 9.9k
Migrate all packages to TypeScript #5817
Copy link
Copy link
Closed
Labels
better engineeringNot a bug or feature requestNot a bug or feature requestproposalThis issue is a proposal, usually non-trivial changeThis issue is a proposal, usually non-trivial change
Metadata
Metadata
Assignees
Labels
better engineeringNot a bug or feature requestNot a bug or feature requestproposalThis issue is a proposal, usually non-trivial changeThis issue is a proposal, usually non-trivial change
Type
Fields
Give feedbackNo fields configured for issues without a type.
💥 Proposal
Our TypeScript coverage now has reached 80%+. Recent attempts:
And chores left:
missingrefactor(core): convert theme-fallback to TS #6511theme-fallbackmissing theme— Thanks @armano2! refactor(plugin-ideal-image): migrate package to TS #5940not migrated at all— Thanks @armano2! refactor(plugin-pwa): migrate package to TS #5941To anyone who ever thinks about this: be prepared. It's just 80 lines, but my three attempts to migrate it all ended up in a pathetic sea of red squiggly lines. Remark plugins are born to be JS, and just let it be. Can't believe I did it for mdx-loader—that's already remarkable enoughThanks @duanwilliam! refactor(remark-plugin-npm2yarn): migrate package to TS #5931not migrated at all— Thanks @armano2! refactor(theme-search-algolia): migrate package to TS #5935Migration should be done after #5612 where we can make the theme transpiled more easily, or after #5816 where we can probably make the two transpilation targets more similar.
Have you read the Contributing Guidelines on issues?
Yes