You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I dropped this rule because it's too complex and incomplete. If we want to enforce some naming conventions with the linter I suggest we use someone else's config and stick to it without exceptions or else we (I) will end up doing this:
Unfortunately the biggest drawback of Sindre Sorhus’ configuration for this rule excludes UPPER_SNAKE_CASE, which you might not like.
Presumably we could find one like Sindre Sorhus and then tweak it for some naming conventions we care about?
The issue is the config is lengthy and it doesn't lend itself to quick fixes, especially because it's hard to figure out what a specific value will and will not allow in practice.
Basically I don’t think it's worth spending time tweaking it manually and frustrate other developers until we find perfection. If possible I'd take the Prettier way and just go with it.
I'll try other configurations on the extension and see what matches our style the most.
I thought that the recommended config included by @typescript-eslint/naming-convention would work for us, but 1085 errors (also no ALL_UPPERCASE allowed)
From https://github.com/pixiebrix/pixiebrix-app/pull/1246#discussion_r811072628
Related:
The text was updated successfully, but these errors were encountered: