Skip to content
This repository has been archived by the owner on Mar 9, 2023. It is now read-only.

Rust Fix WASM #110

Merged
merged 4 commits into from
Mar 24, 2022
Merged

Rust Fix WASM #110

merged 4 commits into from
Mar 24, 2022

Conversation

droogmic
Copy link
Collaborator

No description provided.

Move WASM to it's own crate to keep library simple (in terms of deployment)

./web/ now works in parllel with ./rust/ > trunk serve
@droogmic
Copy link
Collaborator Author

cc: @BudgieInWA

Copy link
Contributor

@dabreegster dabreegster left a comment

Choose a reason for hiding this comment

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

Separate crate makes sense after all. Thanks for cleaning up my mess! I need to actually configure the git hooks on my end. (I'm still used to envs where the presubmit tests are enforced and the commit won't go through)

@droogmic
Copy link
Collaborator Author

Separate crate makes sense after all. Thanks for cleaning up my mess! I need to actually configure the git hooks on my end. (I'm still used to envs where the presubmit tests are enforced and the commit won't go through)

I am disappointed we (software engineering) don't have better standards for sharing pre-commit/pre-push hooks :(

@droogmic droogmic merged commit 39499ad into a-b-street:main Mar 24, 2022
@droogmic droogmic deleted the rust_fix_wasm branch March 24, 2022 21:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants