- From: Domenic Denicola <notifications@github.com>
- Date: Thu, 29 Jul 2021 10:24:44 -0700
- To: heycam/webidl <webidl@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Thursday, 29 July 2021 17:24:56 UTC
@domenic approved this pull request. LGTM with nit > @@ -6830,7 +6830,7 @@ The parameterized type must not be a [=dictionary type=], [=sequence type=], or Similar to [=sequence types=] and [=frozen array types=], observable array types wrap around ECMAScript array types, imposing additional semantics on their usage. -Observable array types must only be used as the type of [=regular attributes=]. +Observable array types must only be used as the type of [=regular attributes=] defined on the [=interface=]. ```suggestion Observable array types must only be used as the type of [=regular attributes=] defined on an [=interface=]. ``` -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/heycam/webidl/pull/1001#pullrequestreview-718353160
Received on Thursday, 29 July 2021 17:24:56 UTC