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

WebAssembly Housekeeping/streams #1347

Closed
wants to merge 0 commits into from
Closed

Conversation

insipx
Copy link
Contributor

@insipx insipx commented Nov 27, 2024

  • Create a common crate to de-duplicate some test util functions (rand_{vec|string|array|hexstring|etc})
  • common crate now houses wasm shims to std::time/sleep. Helps discoverability and reduces dependencies on xmtp_mls
  • common crate houses the retry module, so that it may be re-used in other crates w/o depending on xmtp_mls.
    • this could make it easier to further separate API client code from xmtp_mls in the future if we want to
  • rand_{vec|string|buffer} now allow specifying vector length
  • tests which were previously only running in native now run in wasm (xmtp_id/consent sync + message history tests)
  • some #[cfg()] cleanup

@insipx insipx changed the title WebAssembly Housekeeping WebAssembly Housekeeping/streams Nov 27, 2024
@insipx insipx force-pushed the insipx/browser-streaming branch from befcac9 to 11a08da Compare November 30, 2024 16:24
@insipx insipx closed this Dec 3, 2024
@insipx insipx force-pushed the insipx/browser-streaming branch from 11a08da to f6f9af9 Compare December 3, 2024 20:35
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