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

TAG review request: WebAssembly Threads #299

Closed
3 of 5 tasks
binji opened this issue Aug 17, 2018 · 2 comments
Closed
3 of 5 tasks

TAG review request: WebAssembly Threads #299

binji opened this issue Aug 17, 2018 · 2 comments
Assignees

Comments

@binji
Copy link

binji commented Aug 17, 2018

Bonjour TAG,

I'm requesting a TAG review of:

Further details (optional):

You should also know that...

[please tell us anything you think is relevant to this review]

We'd prefer the TAG provide feedback as (please select one):

  • open issues in our Github repo for each point of feedback
  • open a single issue in our Github repo for the entire review
  • leave review feedback as a comment in this issue and @-notify [github usernames]
@travisleithead travisleithead self-assigned this Sep 10, 2018
@plinss plinss added this to the 2018-10-02-telcon milestone Sep 18, 2018
@travisleithead
Copy link
Contributor

The TAG reviewed this today and while we're not deep experts in Atomics, we appreciated the well-written explainer that helped us get a pretty good idea of what was being proposed, how it might work, and what new primitives are being introduced. Thank you so much for the explainer--we can't stress enough how much we love seeing a well-written explainer (spread the word!). We also noted the Security and Privacy review and agree with your assessments. We appreciated the use of Workers and (generally) leveraging the existing platform to build-up the WASM threading model. Thanks for reviewing with us!

@binji
Copy link
Author

binji commented Oct 2, 2018

Glad to hear the explainer helped. And thank you for the review!

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

No branches or pull requests

5 participants