Pelle Wessman

Results 694 comments of Pelle Wessman
trafficstars

> `import/no-unused-modules` is useful and should be prioritized. Not sure if that rule even belongs in a linter or if a separate tool like knip is a better idea: https://knip.dev/

👍 If anything I think that issue should be a duplicate of this one as `clearer types` is a somewhat ambiguous request 😜

Yeah, a generic creator function could potentially go far and I would also explore the composability of types and whether it would be feasible to set up and maintain something...

One potentially low hanging fruit by the way would be to modify eg `MeltElement` and give it default types: ```ts export type MeltElement< S extends Stores | undefined = undefined,...

Right, I see now, yeah I think providing defaults is pretty common and makes it more discoverable as then one can find that type simply by drilling deeper into the...

Somewhat related proposal: https://github.com/microsoft/vscode/issues/224581

Yeah, eg [`type-coverage`](https://github.com/plantain-00/type-coverage) checks for the use of `any` I would be mostly interested in knowing whether all exports of a file at all has been directly imported/used from a...

If one looks for all symbols exported from every non-test file and compares that to every symbol used within a `test()`, then that could possibly be a first step.

This is probably a breaking change? And maybe should be optional?

Additional links: * https://github.com/modernweb-dev/web/blob/HEAD/docs/guides/test-runner/getting-started.md * https://modern-web.dev/docs/test-runner/test-frameworks/mocha/ * https://modern-web.dev/guides/test-runner/browsers/#testing-in-all-evergreen-browsers * https://modern-web.dev/docs/test-runner/testing-in-a-ci/ * https://modern-web.dev/docs/test-runner/commands/ * https://github.com/coryrylan/web-test-runner-performance