Add 'recommended' status to each rule as a starter

[jdforrester]

Now-recommended rules:
- no-animate-toggle
- no-animate
- no-class-state
- no-constructor-attributes
- no-deferred
- no-extend
- no-fx-interval
- no-global-selector
- no-jquery-constructor
- no-load-shorthand
- no-on-ready
- no-parse-html-literal
- no-ready
- no-sizzle
- variable-pattern

Fixes #249.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (6f5cc33) to head (aff0669).

Additional details and impacted files

@@            Coverage Diff            @@
##            master      #317   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          103       103           
  Lines          603       603           
=========================================
  Hits           603       603           

Files	Coverage Δ
src/rules/no-ajax-events.js	100.00% <ø> (ø)
src/rules/no-animate-toggle.js	100.00% <ø> (ø)
src/rules/no-animate.js	100.00% <ø> (ø)
src/rules/no-append-html.js	100.00% <ø> (ø)
src/rules/no-class-state.js	100.00% <ø> (ø)
src/rules/no-constructor-attributes.js	100.00% <ø> (ø)
src/rules/no-deferred.js	100.00% <ø> (ø)
src/rules/no-extend.js	100.00% <ø> (ø)
src/rules/no-fx-interval.js	100.00% <ø> (ø)
src/rules/no-global-selector.js	100.00% <ø> (ø)
... and 9 more

[edg2s]

How have you decided which rules are recommended?

[jdforrester]

How have you decided which rules are recommended?

Gut feeling / commonality in practice of changes, mostly. Very happy to change any of them.

[edg2s]

Is the boolean used for anything? I'm hesitant to include this in the release without thinking about it a bit more.

[jdforrester]

Is the boolean used for anything? I'm hesitant to include this in the release without thinking about it a bit more.

The next commit in the stack requires this commit.

[jdforrester]

[Edit: Oh, never mind, you just opted out for now. Sure.] Wait, now you've squash-merged a different version of this commit in 8acaa2e instead?

[edg2s]

According to the docs:

recommended: (boolean) For core rules, this specifies whether the rule is enabled by the recommended config from @eslint/js.

We also have a recommended config that only applies variable-pattern. If we do use the flag it should match the list. I'm not sure why you would specify that a rule is recommended in two places though...

[edg2s]

Not also in general for the parent property docs:

docs: (object) Properties often used for documentation generation and tooling. Required for core rules and optional for custom rules. Custom rules can include additional properties here as needed.

https://eslint.org/docs/latest/extend/custom-rules