[cashtab-components] Running npm upgrade
ClosedPublic
Actions

Authored by bytesofman on Sep 19 2022, 18:01.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABCffede807dc64: [cashtab-components] Running npm upgrade

Summary

T2678

Depends on D12004

Run npm upgrade to get ejs > 3.1.7

Test Plan

npm run build and no errors
npm run storybook , starts up, buttons interact with extension

Diff Detail

Repository

rABC Bitcoin ABC

Branch

components-dep-upgrades

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 20063
Build 39813: Build Diff	cashtab-components-tests
Build 39812: arc lint + arc unit

Event Timeline

bytesofman created this revision.Sep 19 2022, 18:01

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 19 2022, 18:01

bytesofman requested review of this revision.Sep 19 2022, 18:01

Harbormaster completed remote builds in B20063: Diff 35007.Sep 19 2022, 18:03

bytesofman added a child revision: D12006: [cashtab-components] upgrading jest.Sep 19 2022, 18:18

run build and run storybook tested all ok, however I noticed in package-lock.json the npm upgrade picked up 3.1.8 for ejs, not 3.1.7. Any issues with this versioning?

In D12005#268615, @emack wrote:

run build and run storybook tested all ok, however I noticed in package-lock.json the npm upgrade picked up 3.1.8 for ejs, not 3.1.7. Any issues with this versioning?

Any version 3.1.7 or greater resolves the issue, so 3.1.8 is fine

emack accepted this revision.Sep 20 2022, 14:32

This revision is now accepted and ready to land.Sep 20 2022, 14:32

Closed by commit rABCffede807dc64: [cashtab-components] Running npm upgrade (authored by bytesofman). · Explain WhySep 20 2022, 21:46

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCffede807dc64: [cashtab-components] Running npm upgrade.