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

Refactor MetaMask extension to TypeScript #13452

Open
brad-decker opened this issue Jan 28, 2022 · 0 comments
Open

Refactor MetaMask extension to TypeScript #13452

brad-decker opened this issue Jan 28, 2022 · 0 comments

Comments

@brad-decker
Copy link
Contributor

Background

It has been a long term goal of the MetaMask engineering team to convert all organization JavaScript code to TypeScript to take advantage of it's developer experience and type safety. Our goal was to convert all of the extension dependencies that we maintain to TypeScript first so that we could rely upon the types we create for them. We have progressed that goal a great deal, with only a handful of repositories remaining that have not yet been converted. Our new strategy is to try and convert leaf node files with few dependencies to use TypeScript, and then work our way to the trunk files. If we hit files that bring in dependencies that are not yet typed we will resolve to find types for them in the @types DefinitelyTyped repo. If those do not exist, or we own the dependency that is untyped, we will work around them until we have types.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants