- From: Domenic Denicola <notifications@github.com>
- Date: Thu, 06 Aug 2020 12:23:32 -0700
- To: heycam/webidl <webidl@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Thursday, 6 August 2020 19:23:44 UTC
@domenic commented on this pull request. Wow, this turned out simpler than I expected. I guess the actual change, e.g. allowing it in unions, falls out of changing the grammar? Let's fold in Type into ReturnType. Then we should make sure we're ready for a big spec update of this magnitude before merging: - File issues on all the places located in https://github.com/heycam/webidl#filing-issues-elsewhere, pointing to this about-to-be-merged PR. - @saschanaz, can you get your bot ready to pull request all the specs? - /cc @foolip and @stephenmcgruer who has been doing some IDL stuff, I think related to reffy reports. (Maybe reffy should also go in that list of places to file issues?) > @@ -2520,7 +2525,6 @@ steps are:” followed by a list, or “The <code>|operation|(<var ignore>arg1</ <pre class="grammar" id="prod-ReturnType"> ReturnType : Type Let's fold it away; I think a big win of this PR is unifying the two value spaces. -- 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/906#pullrequestreview-462807357
Received on Thursday, 6 August 2020 19:23:44 UTC