-
Notifications
You must be signed in to change notification settings - Fork 47
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
New principle: Patterns for x() vs xSync() #402
Comments
I think both naming schemes are useful in different cases, basically |
Note that sometimes the common case and the case you want to encourage may be different.
Might be a good idea to wait for us to at least discuss, as right now that's just my personal opinion and not TAG consensus. We can add it to the agenda for next week, so you'd only need to wait a few days. |
@LeaVerou your personal opinion was well received in the WG and helped us reach consensus. That said, we’ll track this issue for any further guidance. For now, we settled on |
We discussed today and agreed to write a temporary principle. |
* Add new principle for sync() vs async(). Resolves #402. * Update index.bs Co-authored-by: Lea Verou <[email protected]> * Update index.bs Co-authored-by: Lea Verou <[email protected]> * Update index.bs Co-authored-by: Lea Verou <[email protected]> * Update index.bs Co-authored-by: Peter Linss <[email protected]> * Update index.bs * Update index.bs * Update index.bs * Update index.bs * Apply suggestions from code review --------- Co-authored-by: Lea Verou <[email protected]> Co-authored-by: Peter Linss <[email protected]>
* Add new principle for sync() vs async(). Resolves #402. * Update index.bs Co-authored-by: Lea Verou <[email protected]> * Update index.bs Co-authored-by: Lea Verou <[email protected]> * Update index.bs Co-authored-by: Lea Verou <[email protected]> * Update index.bs Co-authored-by: Peter Linss <[email protected]> * Update index.bs * Update index.bs * Update index.bs * Update index.bs * Apply suggestions from code review --------- Co-authored-by: Lea Verou <[email protected]> Co-authored-by: Peter Linss <[email protected]>
This was brought up in our discussion of w3ctag/design-reviews#771.
TL:DR; we don't have guidance on what to do when an API wants to be synchronous, and the very small subset of cases where this is valid (for the time being).
The text was updated successfully, but these errors were encountered: