Re: [heycam/webidl] Add missing terminals to Other and ArgumentNameKeyword (#813) (#814)

You mentioned on #815:

> At the very least, we should import that code into this repo. I wouldn't mind seeing it rewritten in python either, but that's not urgent.

I’d be happy to have a go at both integrating the postprocessor code and adding a test to check-grammar\* that confirms that Other describes the set of all terminals save the seven intended exceptions. Even if they don’t share code directly, they would benefit from a bit more uniformity since they are doing very similar things. However I’d like to do so as part of a more general effort to clean the build and test scripts and, in particular, to make them platform agnostic.

Aside from the deploy script, the platform assumptions currently made seem to be avoidable. I’m on windows, so I already had to write a js equiv of the makefile to jump the initial barrier to participation, and I could polish/share that.

I’m afraid though that if I do this, stuff will either remain JS or, in the base of bash stuff, _become_ JS 😱. My Python is pretty limited.

\* I now see this script repeats the "float" issue from #815, which might hint that there are additional worthwhile grammar checks that are missing.

-- 
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/814#issuecomment-539734240