[New] forward-ref-uses-ref: add rule for checking ref parameter is added

[NotWoods]

Closes #3158

Adds a new rule to require that a second parameter is used with forwardRef. Offers suggestions to add the parameter or remove the forwardRef wrapper.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.76%. Comparing base (597553d) to head (ad8d26e).
Report is 5 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3667      +/-   ##
==========================================
+ Coverage   97.62%   97.76%   +0.13%     
==========================================
  Files         132      136       +4     
  Lines        9692     9743      +51     
  Branches     3520     3536      +16     
==========================================
+ Hits         9462     9525      +63     
+ Misses        230      218      -12     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ljharb]

There’s a few lines that are uncovered; it’d be great to add tests that cover them.

[ljharb]

(there's still some uncovered lines)

[NotWoods]

I'll work on this more in January; currently busy with holidays. Will make sure to get the last few lines covered and see if I can figure out why ESLint 6 & 7 are failing.

[ljharb]

There's still a few uncovered added lines.

[NotWoods]

I don't see the uncovered lines in Codecov, can you help point them out to me?

[ljharb]

@NotWoods they're right in the "files changed" tab - see https://github.com/jsx-eslint/eslint-plugin-react/pull/3667/files#diff-25114be5f5c5bf7d89a5cc5dfb905ca0d6aafda4a7cb7b49d20df89ceea52bf4R72 for an example

[ljharb]

Rebased; tests are failing.

[ljharb]

rebased; LGTM assuming tests pass