Next.js Eslint

[michalsanger]

Next.js v11 integrates Eslint:
https://nextjs.org/docs/basic-features/eslint

Should we include their rules?
https://nextjs.org/docs/basic-features/eslint#eslint-plugin

[tbergquist-godaddy]

Should we include their rules?

Yeah, I think we should do that, we are using nextjs for all our apps, so that makes sense.

[mrtnzlml]

I've actually started working on this a long time ago when it was in a beta phase. I will be happy to hand this PR over for someone to take over. One issue I encountered is that we have to somehow make this feature optional because not everything is Next.js. I remember that there were some rules/errors which didn't make any sense in a larger scope as we have the config now. I am pretty sure that was the reason why I didn't finish it yet. 🤔

Do you want to work on it?

[michalsanger]

@mrtnzlml something like this #2569

[mrtnzlml]

Nice! I will try to check it as soon as I can. Please, be patient. 🐌

[mrtnzlml]

I updated the original PR: #2562

We might need to rethink it a little bit. It's definitely configurable from outside of Universe, however, how do I disable Next.js rules for one repo inside Universe? 🤔

[michalsanger]

My idea was to do it via overrides in .eslintrc.js

[mrtnzlml]

Will be released in version 6.3.0 (https://github.com/adeira/eslint-config-adeira/blob/39f2192d5596473cb6ff9c65068747e94e3abc03/CHANGELOG.md#unreleased)