- 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