ChainSafe / lodestar

🌟 TypeScript Implementation of Ethereum Consensus
https://lodestar.chainsafe.io
Apache License 2.0
1.19k stars 297 forks source link

Migrate missing eslint rules during migration from eslint #7150

Open nazarhussain opened 1 month ago

nazarhussain commented 1 month ago

Problem description

Based on the discussion https://github.com/ChainSafe/lodestar/pull/7108#issuecomment-2386606996 there are few eslint rules which don't have alternative biome rules yet. We keep this issue as tracker to migrate those when available, or decide which rules are not useful anymore.

Solution description

There should be no useful rule left out.

Additional context

No response

nazarhussain commented 1 month ago

The semi rule is actually a formatting configuration in biomejs, not the lint rule.

https://github.com/ChainSafe/lodestar/blob/a570048187f1e729c0be3939239fcfa3aa0a008e/biome.jsonc#L266