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

use xxhash-wasm #8

Merged
merged 3 commits into from
May 12, 2021
Merged

use xxhash-wasm #8

merged 3 commits into from
May 12, 2021

Conversation

shannonwells
Copy link
Collaborator

The xxhash library depends on node. We need something that can be run in a browser. This is a WebAssembly version, which required us to use async functions.

@shannonwells shannonwells force-pushed the feat/use-xxhash-wasm branch from 22829dd to 873ae97 Compare May 11, 2021 16:51
@shannonwells shannonwells marked this pull request as ready for review May 12, 2021 22:26
* update tests due to new xxhash-wasm package
* remove reference to old xxhash
@shannonwells shannonwells force-pushed the feat/use-xxhash-wasm branch from 53bd38c to 4c9094d Compare May 12, 2021 22:30
@shannonwells shannonwells enabled auto-merge (squash) May 12, 2021 22:31
@shannonwells shannonwells merged commit 4099f37 into main May 12, 2021
@shannonwells shannonwells deleted the feat/use-xxhash-wasm branch May 12, 2021 22:45
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