-
Notifications
You must be signed in to change notification settings - Fork 89
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update proposal to match new group consensus #293
Conversation
Do we have these patterns (relation patterns) in this PR from @rbuckton? when (x > 20 and < 40) ... |
No, they don't have champion consensus yet. |
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
…. Since many return failure as non-false anyway, it often won't be usable anyway.
Hi @tabatkins did you forget the when pattern: ...
if (expression): ...
default: ... |
Co-authored-by: Jordan Harband <ljharb@gmail.com>
Co-authored-by: Jordan Harband <ljharb@gmail.com>
Co-authored-by: Jordan Harband <ljharb@gmail.com>
Co-authored-by: Jordan Harband <ljharb@gmail.com>
Co-authored-by: Jordan Harband <ljharb@gmail.com>
Co-authored-by: Jordan Harband <ljharb@gmail.com>
Co-authored-by: Jordan Harband <ljharb@gmail.com>
The new spec (#295) is ready for review, I hope you can pick some ideas from it and update the README because more details are considered when writing the spec than just considering the normal cases. thanks! |
…ger have to special-case things like Infinity
… to their own subsection so I can define the strict array-like behavior.
…d pattern in the example. Remove the issue.
WORK IN PROGRESS, DO NOT MERGE
Rewriting the proposal to be in line with https://gist.github.com/tabatkins/51f35f88d7eea61d9ecbe3e82da817a5, as agreed by the champion group.
Currently I've only got the matchers rewritten as a first pass; have not yet done the
match(){}
expression or theis
operator.Closes #314.
Closes #313.
Closes #303.
Closes #301.
Closes #299.
Closes #297.
Closes #283.
Closes #282.
Closes #281.
Closes #264.
Closes #255.
Closes #253.