chore: move lint rules to Biome

[ematipico]

Changes

This PR turns off some eslint rules, and turns on the relative Biome rules.

The noUnsedVariables rule reports information diagnostics, so it should not bother too much. I run its respective code action (unsafe), which renamed all unused variables with an underscore e.g. el -> _el.

The no-console rule from eslint was incorrectly configured, because I ended removing a lot of suppression comments.

I also moved the configuration file to JSONC, so I could move the comments we had in the eslint configuration file.

Testing

Current CI should work

Docs

N/A

[changeset-bot]

⚠️ No Changeset found

Latest commit: 9664125

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[bluwy]

Is the rename from console.log to console.info intentional? Seems to have changed for many other code too.

[ematipico]

Many of these logs had suppression comments (many), this means those logs were intentional. console.log are usually for debugging purposes, but console.info are usually on purpose.

I deduced that all those logs were actually intentional, which means it makes more sense to turn them into .info.

What do you think? Does it make sense? Do you agree?

[bluwy]

That makes sense to me. I also noticed the PR doesn't really change to .info a lot, only for this performance/ directory which we don't use much either. So I think this is fine by me.