feat(eslint-plugin): add new rule no-require-imports

[lukyth]

Adds the equivalent of TSLint's no-require-imports rule.

[codecov]

Codecov Report

Merging #199 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #199      +/-   ##
==========================================
+ Coverage   96.78%   96.78%   +<.01%     
==========================================
  Files          55       56       +1     
  Lines        2516     2522       +6     
  Branches      372      372              
==========================================
+ Hits         2435     2441       +6     
  Misses         42       42              
  Partials       39       39

Impacted Files	Coverage Δ
...ages/eslint-plugin/lib/rules/no-require-imports.js	100% <100%> (ø)

[lukyth]

@bradzacher Thank you for your review :) I've addressed all the comments. Could you please check it out?

[j-f1]

Just thinking — would this be better suited as a rule in eslint-plugin-import?

[lukyth]

Not sure what's best here. I can try moving this over to eslint-plugin-import if you think it's more appropriate.

[sindresorhus]

I think the rule should stay here. eslint-plugin-import is not very actively maintained and it also doesn't currently have any rules that target TypeScript.

[JamesHenry]

Apologies for the conflicts, we decided to remove the counts from the ROADMAP here: #225

They are too awkward to maintain manually, so this is the last time you will have to deal with syncing that up!