search

ponylang / rfcs

RFCs for changes to Pony
https://ponylang.io/
60 stars 48 forks source link

Parameter assignment syntax #174

Closed jemc closed 1 year ago

jemc commented 4 years ago

This PR adds my RFC for assign param syntax.

aturley commented 4 years ago

Discuss more next week.

SeanTAllen commented 1 year ago

@jemc how would you feel about moving forward with this. instead of @? I'd like to move this and #173 forward but we have some syntax issues to resolve. I'm in favor of "this" because it is the least breaking option.

jemc commented 1 year ago

I'd agree that this. is "better than nothing" for the purposes of this RFC, so I'd be good with amending this RFC to use that syntax. It's still less boilerplate than the current status quo.

For #173 I would not be in favor of making that change with this. - I think it's too much friction added for something so ubiquitous.

SeanTAllen commented 1 year ago

We appear to have an impasse on #173, but it would be good to move forward with this RFC so let's do that and continue to discuss #173.

jasoncarr0 commented 1 year ago

One more note possibly worth adding: are any names supported for keyword argument syntax? Does that create semver issues with the name requirements?

SeanTAllen commented 1 year ago

It was noted on the sync call that @jemc will note that how this interacts with named arguments in unresolved questions.

SeanTAllen commented 1 year ago

This was accepted during sync. I will do the merge and what not later.