Page MenuHomePhabricator

[ecashaddrjs] Remove version number from minified builds
ClosedPublic

Authored by bytesofman on Mar 27 2023, 13:16.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC615249d54706: [ecashaddrjs] Remove version number from minified builds
Summary

T3039

Depends on D13477

To prevent the need for manually updating the README with every release, remove the version number from minified builds. The link will now reference the latest version. Past versions can be accessed by users manually specifying the version in the URL, e.g. https://unpkg.com/ecashaddrjs@<userChangesThis>/dist/cashaddrjs.min.js

Since these builds are no longer included in the git repo, a new npm script npm run prepublish is added. This will ensure that npm run build is always run before npm run publish (and thus that the package is not inadvertently published without first building the latest version).

Now that the build files have the same name, there is no need for the npm run clean script or its dependency shx. These are removed.

Test Plan

npm run build and output files do not have version info. Newly generated file paths match the updated link in README.md.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 22733
Build 45086: Build Diff
Build 45085: arc lint + arc unit

Event Timeline

Removing unused npm dependency

Fabien requested changes to this revision.Mar 27 2023, 15:34
Fabien added a subscriber: Fabien.
web/ecashaddrjs/package.json
14 ↗(On Diff #38922)

This one is also useless now

This revision now requires changes to proceed.Mar 27 2023, 15:34

Removing no longer used script and npm module

This revision is now accepted and ready to land.Mar 27 2023, 16:13