Page MenuHomePhabricator

[cashtab-componets] npm module maintenance
ClosedPublic

Authored by bytesofman on Mar 8 2023, 18:01.

Details

Reviewers
emack
Group Reviewers
Restricted Project
Commits
rABCe43c035c8780: [cashtab-componets] npm module maintenance
Summary

T2979

Running npm audit fix

Note: still have some vulnerabilities related to storybook. This is not used in prod except for examples, so severity is lower.

Test Plan

npm run storybook and test with extension

Diff Detail

Repository
rABC Bitcoin ABC
Branch
cashtab-components-npm-maint
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 22296
Build 44219: Build Diffcashtab-components-tests
Build 44218: arc lint + arc unit

Event Timeline

As a separate issue, cashtab-components and its dependencies needs to be re-evaluated. This has been postponed since the dependencies are not the highest priority improvement for cashtab-components, and use is still minimal.

A 3mb package-lock.json is pretty incredible.

emack requested changes to this revision.Mar 9 2023, 00:52
emack added a subscriber: emack.

It's failing to start for me with the following exception:

image.png (776×1 px, 196 KB)

This is after npm i as well after arc patching.

This revision now requires changes to proceed.Mar 9 2023, 00:52

Probably needs a storybook upgrade as well to deal with the new dependencies. Will check it out.

Upgrading webpack and storybook

yup the webpack and storybook upgrades fixed it.

This revision is now accepted and ready to land.Mar 9 2023, 22:39