Page MenuHomePhabricator

[cashtab] Upgrade node modules
AbandonedPublic

Authored by deadalnix on Jan 7 2021, 00:01.

Details

Reviewers
majcosta
alcipir
Group Reviewers
Restricted Project
Summary

As per title.

Test Plan
Run the app, it still works.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
updatenode
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 14608
Build 29175: Build Diff
Build 29174: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jan 7 2021, 06:02

Rebased

Removed @testing-library/react-hooks because it has conflicts, and it doesn't seems liek it is used. It would be good to double check, because everything seems busted around here.

Shouldn't package.json be changed as well when running npm update?

alcipir requested changes to this revision.Jan 7 2021, 08:03

Running the test suite after installing packages breaks one of the unit tests: npm run test:coverage

This revision now requires changes to proceed.Jan 7 2021, 08:03

I think this diff is now obsolete. Package updates and dependencies have been handled in other diffs, most recently D8905.