Page MenuHomePhabricator

[ecashaddrjs] Copy all files to support prepublish command in CI
ClosedPublic

Authored by bytesofman on Dec 21 2023, 16:07.

Details

Summary

Support autodeploy for ecashaddrjs

ecashaddrjs, unlike ecash-coinselect, must build before deployment -- the browser version is built with webpack, while the nodejs version just uses the existing JS. This is because browsers do not tend to support the Buffer type which is used in the library.

The npm ci command was failing in docker build bc it requires the directory structure to be intact due to the prepublish command. I didn't know this but apparently npm ci will also run npm prepublish.

I was able to repeat the same error msg thrown by teamcity with this testplan, which is corrected with the changes in this diff.

Test Plan

docker build -t ecashaddrjs_local succeeds

Diff Detail

Repository
rABC Bitcoin ABC
Branch
ecashaddrjs-build-fix
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 26044
Build 51662: Build Diffecashaddrjs-tests
Build 51661: arc lint + arc unit

Event Timeline

bytesofman retitled this revision from [ecashaddrjs] Remove prepublish step to support CI deployment to [ecashaddrjs] Copy all files to support prepublish command in CI.Dec 21 2023, 16:10
bytesofman edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Dec 22 2023, 08:32