-
Notifications
You must be signed in to change notification settings - Fork 10
Path to Stage 4! #7
Comments
@ljharb You'll need to add the new alternative in each of the SDOs defined over RelationalExpression. What's there LGTM though. |
Contents lgtm. |
So far this seems fine to me but I'd also want to check it against the base proposal. Is there a rendered version of tc39/ecma262#1668 ? Rendering of that one seems to have failed. |
@waldemarhorwat tc39/ecma262#1668 has now been rebased, thanks to @Ms2ger . You can find a rendered version at https://arai-a.github.io/ecma262-compare/?pr=1668 (thanks, @arai-a!) |
@ljharb I'd be happy to write the Test262 tests. Can you let me know if that's already underway? |
@jugglinmike that would be awesome. I haven’t had time to start yet, and i wasn’t sure what the best way would be wrt the generated tests. |
Cool. I'll let you know when the patch is up for review |
Here's that patch: tc39/test262#2963 |
This proposal is now stage 4! |
Stage 4Stage 3Stage 2Stage 1The text was updated successfully, but these errors were encountered: