shapesecurity / shift-spec

:fast_forward: Shift AST Specification
http://shift-ast.org/
Other
185 stars 18 forks source link

Tracking Proposals #119

Open jamiebuilds opened 7 years ago

jamiebuilds commented 7 years ago

Hi, I talked to @bakkot a bit about this but I wanted to create a more open issue about it.

Shift is really great to work with, but it's limited to completed ECMAScript specs. I was wondering if people would be up for a bit more formality around proposals.

I'm not looking for a bunch of unstable things to be added to the spec, just that there's a place for in progress specs to agree "If the spec is accepted in its current form, this is what the Shift AST would look like" and the acknowledgement that it could change.

A lot of it is obvious, like catch (e) {}catch {}, but stuff like class properties is not.

It would also be really nice if there was a document for all of these and you didn't have to go through all the issues.

michaelficarra commented 7 years ago

:+1: It's never too early to say "if the feature makes it to stage 4 as-is, what would it look like in the Shift spec?". These could even be tracked as PRs.

jamiebuilds commented 7 years ago

Here's the current list of syntax proposals, would you like me to start working my way through them and open up PRs proposing AST changes for them?

jamiebuilds commented 7 years ago

In order to support class fields, could I change ClassElement.method to something like element or value?

michaelficarra commented 7 years ago

FYI, I probably won't be spending any time reviewing the stage 1 and earlier proposals since they're so likely to change in some way. But thanks for sending over the PRs.