-
Notifications
You must be signed in to change notification settings - Fork 124
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgraded frontend-build version to v12 #613
Conversation
Codecov Report
@@ Coverage Diff @@
## master #613 +/- ##
==========================================
+ Coverage 39.69% 39.84% +0.14%
==========================================
Files 108 108
Lines 2194 2186 -8
Branches 586 585 -1
==========================================
Hits 871 871
+ Misses 1238 1230 -8
Partials 85 85
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
…alqamar95/frontend-build-upgrade
Oof - package-lock.json conflicts. Mind rebasing? |
return bTimeSinceEpoch - aTimeSinceEpoch; | ||
} | ||
|
||
sortVerifiedNameRecords = verifiedNameHistory => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was this function unused?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this function was unused & was triggering react/no-unused-class-component-methods
rule, therefore I removed it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good - think we just need to resolve the conflict. (And the sanity check around whether sortVerifiedNameRecords
was in use)
Ticket:
42: Upgrade eslint to v8.x
What changed?
frontend-build
to v12 (Eslint was updated infrontend-build
version resulting in it's version being bumped to v12. This PR updatesfrontend-build
to reciprocate eslint version update)eslint
issues