Skip to content

feat: allow bindgen-cli --split-linked-modules #1092#1443

Merged
drager merged 1 commit intodrager:masterfrom
codeart1st:split-linked-modules
Nov 2, 2024
Merged

feat: allow bindgen-cli --split-linked-modules #1092#1443
drager merged 1 commit intodrager:masterfrom
codeart1st:split-linked-modules

Conversation

@codeart1st
Copy link
Contributor

@codeart1st codeart1st commented Nov 2, 2024

Make sure these boxes are checked! 📦✅

  • You have the latest version of rustfmt installed
$ rustup component add rustfmt
  • You ran cargo fmt on the code base before submitting
  • You reference which issue is being closed in the PR text

✨✨ 😄 Thanks so much for contributing to wasm-pack! 😄 ✨✨

@codeart1st
Copy link
Contributor Author

Copy link
Owner

@drager drager left a comment

Choose a reason for hiding this comment

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

Thank you

@drager drager merged commit ba5fef3 into drager:master Nov 2, 2024
Twey added a commit to linera-io/linera-protocol that referenced this pull request May 29, 2025
## Motivation

When we merged `linera-web` into this repository, I left some relative
paths in `build.bash` that break the build script.

It would be nice to migrate back to `wasm-pack`, a more standard tool in
the Rust world, but so far `wasm-pack` hasn't made a release that
contains the necessary PR
drager/wasm-pack#1443.

## Proposal

Instead of using relative paths, extract the necessary paths from `cargo
metadata`.

## Test Plan

Tested locally. (We should really run `pnpm build` as part of the CI
job, not just `cargo build`.)

## Release Plan

- Nothing to do / These changes follow the usual release cycle.

## Links

- [reviewer
checklist](https://github.com/linera-io/linera-protocol/blob/main/CONTRIBUTING.md#reviewer-checklist)
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