rfcs icon indicating copy to clipboard operation
rfcs copied to clipboard

Add RFC for assign param syntax.

Open jemc opened this issue 4 years ago • 1 comments

This PR adds my RFC for assign param syntax.

Note: This RFC is based on this other RFC and assumes the other one will be accepted when presenting its syntax examples

jemc avatar May 25 '20 18:05 jemc

Discuss more next week.

aturley avatar Jun 16 '20 18:06 aturley

@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.

SeanTAllen avatar Jan 02 '23 22:01 SeanTAllen

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.

jemc avatar Jan 03 '23 15:01 jemc

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.

SeanTAllen avatar Jan 03 '23 15:01 SeanTAllen

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

jasoncarr0 avatar Jan 03 '23 19:01 jasoncarr0

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

SeanTAllen avatar Jan 03 '23 19:01 SeanTAllen

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

SeanTAllen avatar Jan 24 '23 19:01 SeanTAllen