Re: [whatwg/streams] Meta: update contribution guidelines (PR #1301)

@ricea approved this pull request.

lgtm with nit

>  
 When writing examples or notes, JavaScript variables and values are enclosed in `<code>` tags, not in `<var>` tags.
 
-Use abstract operations in the following scenarios:
-
-- To factor out shared logic used by multiple public APIs, or by multiple other abstract operations. _Example: ReadableByteStreamControllerEnqueueChunkToQueue_.
-- To factor out operations that should be called by other specifications. Other specifications do not require checking of arguments, argument parsing, and other invariants; we assume they use abstract operations appropriately, and so we don't need to enforce correctness by throwing an error if not. Thus we often let the public API enforce invariants before calling to an abstract operation that assumes they hold already. _Example: ReadableStreamDefaultControllerClose_.
+Use abstract operations in the following scenarios to factor out shared logic used by multiple public APIs, or by multiple other abstract operations.

Maybe remove the words "in the following scenarios"?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/pull/1301#pullrequestreview-1810442553
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/streams/pull/1301/review/1810442553@github.com>

Received on Tuesday, 9 January 2024 05:51:29 UTC