Turn on no-unused-imports rule and remove all unused imports

[tdonohue]

Description

Small PR which bans unused imports in all classes. We already have LGTM warning when unused imports exist. This officially bans unused imports, to ensure our code is kept small (and easy to tree-shake).

Unused imports will now throw an error when yarn lint is run.

[lgtm-com]

This pull request fixes 15 alerts when merging ea4abe8 into 1c8d538 - view on LGTM.com

fixed alerts:

• 15 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 15 alerts when merging 8427dd6 into 0395480 - view on LGTM.com

fixed alerts:

• 15 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 15 alerts when merging a9b2812 into f782ee9 - view on LGTM.com

fixed alerts:

• 15 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 15 alerts when merging 76c5bdb into e7e049c - view on LGTM.com

fixed alerts:

• 15 for Unused variable, import, function or class

[github-actions]

Hi @tdonohue,
Conflicts have been detected against the base branch.
Please resolve these conflicts as soon as you can. Thanks!

---

This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.

[lgtm-com]

This pull request fixes 15 alerts when merging 4f22aa2 into dc4b4ff - view on LGTM.com

fixed alerts:

• 15 for Unused variable, import, function or class

[atarix83]

thanks @tdonohue

this is a very useful rule.

I've verified the IDE eslint plugin reports the errors properly (see screenshot) and the yarn lint fails if any unused imports are left

[lgtm-com]

This pull request fixes 15 alerts when merging cc86ba5 into 340cc0a - view on LGTM.com

fixed alerts:

• 15 for Unused variable, import, function or class