Skip to content

Comments

SDK-523 Bundle the JS user library#142

Merged
paulyoung merged 5 commits intomasterfrom
paulyoung/js-user-lib-bundle
Nov 20, 2019
Merged

SDK-523 Bundle the JS user library#142
paulyoung merged 5 commits intomasterfrom
paulyoung/js-user-lib-bundle

Conversation

@paulyoung
Copy link
Contributor

Ported from #56. The second commit is new, and uses babel-loader as in https://github.com/microsoft/TypeScript-Babel-Starter.

As far as I can tell, the trade off there is some extra webpack configuration and a dependency, for not creating intermediate files when bundling. However, I'd prefer not to have more that one way of compiling TypeScript, and the npm scripts seem useful, so I'm thinking we should revert that and not use babel-loader.

I'm creating a bundle for both browsers and Node.js. The latter was primarily motivated by the Motoko test suite relying on the JS user library to verify that the IDL works correctly. I'll update Motoko to use this instead of the branch used in #56.

@paulyoung paulyoung requested a review from a team as a code owner November 5, 2019 19:21
@paulyoung paulyoung changed the title Paulyoung/js user lib bundle Bundle the JS user library Nov 5, 2019
This reverts commit eec6a2d.
@paulyoung paulyoung changed the title Bundle the JS user library SDK-523 Bundle the JS user library Nov 6, 2019
Copy link
Contributor

@hansl hansl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving this to fasttrack some of those old PRs.

@paulyoung paulyoung merged commit cdc6732 into master Nov 20, 2019
@paulyoung paulyoung deleted the paulyoung/js-user-lib-bundle branch November 20, 2019 19:21
dfinity-bot added a commit that referenced this pull request Jul 9, 2021
## Changelog for advisory-db:
Branch: main
Commits: [rustsec/advisory-db@cbeaf18e...4792a373](rustsec/advisory-db@cbeaf18...4792a37)

* [`1889bffd`](rustsec/advisory-db@1889bff) prost-types: Timestamp conversion overflow ([RustSec/advisory-db⁠#954](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/954))
* [`76294321`](rustsec/advisory-db@7629432) Assigned RUSTSEC-2021-0073 to prost-types ([RustSec/advisory-db⁠#955](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/955))
* [`a9c31a6e`](rustsec/advisory-db@a9c31a6) Hotfix [RustSec/advisory-db⁠#957](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/957) until we figure out what to do with it ([RustSec/advisory-db⁠#958](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/958))
* [`3533e434`](rustsec/advisory-db@3533e43) Add rust-ammonia/ammonia[RustSec/advisory-db⁠#142](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/142) ([RustSec/advisory-db⁠#956](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/956))
* [`2d60adf5`](rustsec/advisory-db@2d60adf) Assigned RUSTSEC-2021-0074 to ammonia ([RustSec/advisory-db⁠#959](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/959))
* [`730c1e81`](rustsec/advisory-db@730c1e8) Revert "Hotfix [RustSec/advisory-db⁠#957](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/957) until we figure out what to do with it ([RustSec/advisory-db⁠#958](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/958))" ([RustSec/advisory-db⁠#960](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/960))
* [`674daf6f`](rustsec/advisory-db@674daf6) `ark_r1cs_std::mul_by_inverse` generated unsound constraints in versions below `0.3.1` ([RustSec/advisory-db⁠#961](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/961))
* [`4792a373`](rustsec/advisory-db@4792a37) Assigned RUSTSEC-2021-0075 to ark-r1cs-std ([RustSec/advisory-db⁠#962](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/962))
mergify bot pushed a commit that referenced this pull request Jul 21, 2021
## Changelog for advisory-db:
Branch: main
Commits: [rustsec/advisory-db@cbeaf18e...4792a373](rustsec/advisory-db@cbeaf18...4792a37)

* [`1889bffd`](rustsec/advisory-db@1889bff) prost-types: Timestamp conversion overflow ([RustSec/advisory-db⁠#954](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/954))
* [`76294321`](rustsec/advisory-db@7629432) Assigned RUSTSEC-2021-0073 to prost-types ([RustSec/advisory-db⁠#955](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/955))
* [`a9c31a6e`](rustsec/advisory-db@a9c31a6) Hotfix [RustSec/advisory-db⁠#957](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/957) until we figure out what to do with it ([RustSec/advisory-db⁠#958](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/958))
* [`3533e434`](rustsec/advisory-db@3533e43) Add rust-ammonia/ammonia[RustSec/advisory-db⁠#142](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/142) ([RustSec/advisory-db⁠#956](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/956))
* [`2d60adf5`](rustsec/advisory-db@2d60adf) Assigned RUSTSEC-2021-0074 to ammonia ([RustSec/advisory-db⁠#959](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/959))
* [`730c1e81`](rustsec/advisory-db@730c1e8) Revert "Hotfix [RustSec/advisory-db⁠#957](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/957) until we figure out what to do with it ([RustSec/advisory-db⁠#958](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/958))" ([RustSec/advisory-db⁠#960](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/960))
* [`674daf6f`](rustsec/advisory-db@674daf6) `ark_r1cs_std::mul_by_inverse` generated unsound constraints in versions below `0.3.1` ([RustSec/advisory-db⁠#961](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/961))
* [`4792a373`](rustsec/advisory-db@4792a37) Assigned RUSTSEC-2021-0075 to ark-r1cs-std ([RustSec/advisory-db⁠#962](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/962))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants