fix: detect function-style rules exported using a variable

[bmish]

Updates our rules to detect old function-style rules that are defined in variables:

const rule = function(context) { return {} };
export default rule;

const rule = function(context) { return {} };
module.exports = rule;

In particular, this is needed so that our eslint-plugin/prefer-object-rule can detect and automatically convert these old function-style rules to the new object-style format.

Follow-up to:

• fix: detect rules exported using a variable #233
• Breaking: Support ESM rules #177
• Fix: Correctly handle rules that are missing meta or have meta / create defined in variables #225

[aladdin-add]

LGTM, thanks! 💯

[bmish]

Thanks for the review! Side node, I'd love to be considered for becoming an official collaborator on this project if you folks are open to it @not-an-aardvark. Looking forward to helping out a lot more.

[aladdin-add]

ah, I was thinking the same. But I don't have the admin access too. /_ \

@not-an-aardvark can you give me the admin access, so I can add @bmish, and some more (e.g. rename default branch to main).

[not-an-aardvark]

I added @bmish to the repository and renamed the default branch to main. Let me know if there are other things I should do.

I'm not sure it's possible for me to give admin access because this is a user-owned repository. I think we would need to transfer the repository to an organization in order to do that. I would be fine with transferring it to an organization if that makes things easier.

[aladdin-add]

great, thanks! 👍

nicholas has a plan to make an eslint-community org. this one sounds a good fit. I'll ping you when everything is ready. :)

[bmish]

Thanks! Yeah, definitely in favor of this being part of the future eslint-community org (I suggested that here too: eslint/eslint#14884 (comment)).