Upgrade eslint to v8.x

[BilalQamar95]

Description

Ticket
42: Upgrade eslint to v8.x

Description:

• Upgraded eslint to v8.18 along with @edx/eslint-config
• Resolved linting issues post upgrade.

Merge Checklist

• If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
• Does your change adhere to the documented style conventions?
• Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
• Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
• Were your changes tested in the example app?
• Is there adequate test coverage for your changes?
• Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

• Verify your changes were released to NPM at the expected version.
• If you'd like, share your contribution in #show-and-tell.
• 🎉 🙌 Celebrate! Thanks for your contribution.

[netlify]

✅ Deploy Preview for paragon-openedx ready!

Name	Link
🔨 Latest commit	7e93f18
🔍 Latest deploy log	https://app.netlify.com/sites/paragon-openedx/deploys/6332bc01b333170008d90381
😎 Deploy Preview	https://deploy-preview-1421--paragon-openedx.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[codecov]

Codecov Report

Base: 91.62% // Head: 90.40% // Decreases project coverage by -1.21% ⚠️

Coverage data is based on head (c223aa7) compared to base (adf708d).
Patch coverage: 72.01% of modified lines in pull request are covered.

❗ Current head c223aa7 differs from pull request most recent head 7e93f18. Consider uploading reports for the commit 7e93f18 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1421      +/-   ##
==========================================
- Coverage   91.62%   90.40%   -1.22%     
==========================================
  Files         208      208              
  Lines        3547     3534      -13     
  Branches      824      824              
==========================================
- Hits         3250     3195      -55     
- Misses        282      324      +42     
  Partials       15       15              

Impacted Files	Coverage Δ
src/Alert/index.jsx	87.50% <0.00%> (-9.87%)	⬇️
src/Button/index.jsx	73.33% <0.00%> (-26.67%)	⬇️
src/Card/CardStatus.jsx	100.00% <ø> (ø)
src/Collapsible/CollapsibleAdvanced.jsx	88.57% <ø> (ø)
src/Collapsible/CollapsibleVisible.jsx	100.00% <ø> (ø)
src/Collapsible/index.jsx	69.23% <ø> (ø)
src/DataTable/CardView.jsx	20.00% <0.00%> (-7.28%)	⬇️
src/DataTable/SidebarFilters.jsx	16.66% <0.00%> (-8.34%)	⬇️
src/DataTable/TableFilters.jsx	30.00% <0.00%> (-10.00%)	⬇️
src/Dropdown/deprecated/index.jsx	94.84% <ø> (ø)
... and 105 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[adamstankiewicz]

LGTM

[adamstankiewicz]

@BilalQamar95 When you merge this, let's use a chore commit as I don't think it'll be necessary to trigger a new release for these eslint fixes?

[edx-semantic-release]

🎉 This PR is included in version 20.12.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀