Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

examples: trunk-workspace: update for future compat #742

Merged
merged 5 commits into from
Nov 18, 2024
Merged

examples: trunk-workspace: update for future compat #742

merged 5 commits into from
Nov 18, 2024

Conversation

ipetkov
Copy link
Owner

@ipetkov ipetkov commented Nov 18, 2024

Checklist

  • added tests to verify new behavior
  • added an example template or updated an existing one
  • updated docs/API.md (or general documentation) with changes
  • updated CHANGELOG.md

We already have a test which hard codes which version of
wasm-bindgen-cli we're using, no need to have two
The examples were already hard coding a concrete version of
wasm-bindgen-cli so having the extra input was both useless and
confusing
Looks like newer versions of trunk expect the current directory to be
the crate to compile, so adjust the example to take that into account
@ipetkov ipetkov enabled auto-merge (rebase) November 18, 2024 21:35
@ipetkov ipetkov merged commit 9132c6d into master Nov 18, 2024
19 checks passed
@ipetkov ipetkov deleted the trunk branch November 18, 2024 21:37
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.

1 participant