chore: update eslint config to allow direct reference of static class functions

[cgrindel]

Changes

Update eslint configuration to allow static class functions to be referenced directly without wrapping them in an arrow function.

Context

Slack discussion

If someone defines a static comparison function on a class, it would be ideal to reference this function in Array.sort() like so:

bzlmodVersions.sort(BzlmodVersion.defaultCompare);

However, eslint's unbound-method rule will complain with the following message:

@typescript-eslint/unbound-method: Avoid referencing unbound methods which may cause unintentional scoping of `this`.If your function does not access `this`, you can annotate it with `this: void`, or consider using an arrow function instead.

Given that static class functions should be written without a reference to this, configuring ignoreStatic for the rule would allow one to reference the static function directly.

Documentation (please check one with an [x])

• I have updated the documentation, or
• No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

• Code inspection only, or
• Newly added/modified unit tests, or
• No unit tests but ran on a real repository, or
• Both unit tests + ran on a real repository

[cgrindel]

CC: @viceice

[renovate-release]

🎉 This PR is included in version 35.57.0 🎉

The release is available on:

• GitHub release
• 35.57.0

Your semantic-release bot 📦🚀