Get rid of eslint GH workflow

[yuvipanda]

This is handled by prettier in the pre-commit workflow instead

[manics]

I thought prettier only formats code, whereas eslint also detects some likely errors?

[yuvipanda]

Ah whoops I had thought we were using eslint with a much more relaxed rulset, but we aren't!

https://prettier.io/docs/en/comparison.html

Code-quality rules: eg no-unused-vars, no-extra-bind, no-implicit-globals, prefer-promise-reject-errors…

Prettier does nothing to help with those kind of rules. They are also the most important ones provided by linters as they are likely to catch real bugs with your code!

In other words, use Prettier for formatting and linters for catching bugs!