MetaMask / MetaMask/object-multiplex
`@types/readable-stream` should be a dependency, not a dev dependency
Open
Nobody has claimed this yet.
team-wallet-framework
wf-tech-debt
- Dominant language
- TypeScript
- Stars
- 15
- Forks
- 14
- PR merge metrics
- No merged PRs in 30d
Description
Otherwise TypeScript consumers are forced to add this package as a dependency, for instance: https://github.com/MetaMask/swaps-controller/pull/333/files#r1829249847
Contributor guide
No contributing guide indexed for this repository
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.
Research direction
No file or test is named. Inspect the package dependency declarations and the linked consumer example first; done means @types/readable-stream is published as a runtime dependency so TypeScript consumers do not need to install it directly.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- build-system
- Issue type
- Bug
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100