search

matrix-org / matrix-rust-sdk-crypto-nodejs

Apache License 2.0
3 stars 5 forks source link

v0.1.0-beta.7 #9

Closed AndrewFerr closed 1 year ago

AndrewFerr commented 1 year ago

Hrm, publishing to NPM failed apparently due to an upload size limit:

npm ERR! 413 Payload Too Large - PUT https://registry.npmjs.org/@matrix-org%2fmatrix-sdk-crypto-nodejs - Payload Too Large

@poljar Any idea of how to address this?

AndrewFerr commented 1 year ago

Ah, the release workflow included debug files in its tarball for some reason. For comparison, the previous release has only 8 files, while the draft release created by the workflow has 2195.

AndrewFerr commented 1 year ago

Oh, I may have figured it out: under the matrix-rust-sdk repo, the default build profile excluded debug info, but in this repo, it includes it. Updating the release workflow to use npm run release-build ought to put a stop to that.

AndrewFerr commented 1 year ago

The other missing piece was to update .npmignore, which after the repo split, needs to cover a few more files/directories that now get placed in the project root.