- From: Tobie Langel <notifications@github.com>
- Date: Tue, 17 Oct 2017 18:36:16 +0000 (UTC)
- To: heycam/webidl <webidl@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Tuesday, 17 October 2017 18:36:40 UTC
> LGTM, but do you think it's a hazard having the grammar post-processing in two places, one in an npm package and one in a <script class="remove"> in the document? It seems a bit better to me to just remove the dynamic script thing and require Node.js for building. That was my initial thought too. But this script only changed once since it was introduced. I think we can keep it as such for the time being, and maybe by the time it changes again, Bikeshed will have BNF support. -- 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/460#issuecomment-337326390
Received on Tuesday, 17 October 2017 18:36:40 UTC