Milos Djermanovic

Then we probably should fix it, since it will be broken with Espree 8.

Prepared a quick fix: https://github.com/eslint/eslint/pull/14748

I added this to v9.0.0 project. > maybe just add a deprecation warning in v8.0.0 That would be throwing an error in RuleTester? It already throws on any access to...

Prepared https://github.com/eslint/eslint/pull/14769 to throw an error in RuleTester if `SourceCode#getComments()` was called, starting from ESLint v8.0.0. The remaining task is to completely remove `SourceCode#getComments()` in ESLint v9.0.0.

Reopening since it isn't completed (there's still a task to remove the method in v9.0.0).

Hi @marcelschork, thanks for the issue! > So the rule should not only ignore uppercase functions names but also the lowercase constructor property when the option ignoreConstructors is set. Makes...

@eslint/eslint-tsc thoughts about this?

It would be best to add a generic option that accepts a regex. Properties whose name matches the regex and value is a function would be ignored. This would avoid...

> Do you have a proposal for that new option’s name? Maybe `functionsIgnorePattern`?

`methodsIgnorePattern` sounds good to me.