MetaMask / MetaMask/metamask-extension
Remove un-necessary word lists from bundle - replace ethers with submodules
@adonesky1 is already working on this.
Since Aug 9, 2022.
- Dominant language
- TypeScript
- Stars
- 13.2k
- Forks
- 5.6k
- Avg merge
- 2d 5h
- Merged PRs (30d)
- 451
Description
> We are including huge list of words of which only english is being used:
>
> https://output.circle-artifacts.com/output/job/0b87725e-d9fa-4b1e-87c8-209ba5bb939a/artifacts/0/build-artifacts/source-map-explorer/common-0.html
>
> https://output.circle-artifacts.com/output/job/0b87725e-d9fa-4b1e-87c8-209ba5bb939a/artifacts/0/build-artifacts/source-map-explorer/common-6.html
>
> We can get rif of other languages to reduce bundle size.
This ticket is now about replacing `ethers` with submodules throughout our dependency graph.
ongoing related PRs:
- https://github.com/MetaMask/metamask-extension/pull/15461
- https://github.com/MetaMask/metamask-extension/pull/15818
- https://github.com/ethereum-optimism/optimism/pull/3363
- https://github.com/MetaMask/controllers/pull/907
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.