-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Add HashLiteral#has_key?
and NamedTupleLiteral#has_key?
#14890
Add HashLiteral#has_key?
and NamedTupleLiteral#has_key?
#14890
Conversation
0443c58
to
6c29a7a
Compare
ac66a97
to
523d482
Compare
523d482
to
293da0b
Compare
293da0b
to
e5f1187
Compare
Thanks for the PR! In the future, as called out in https://github.com/crystal-lang/crystal/blob/master/CONTRIBUTING.md#making-good-pull-requests, could you not force push? Doesn't matter too much in this case as it was a WIP branch, but it can make the review process harder if force pushes happen with pending comments and such. We squash merge PRs so no need to worry about having a lot of commits. |
Co-authored-by: Johannes Müller <[email protected]>
Co-authored-by: Johannes Müller <[email protected]>
has_key?
to HashLiteral
and NamedTupleLiteral
HashLiteral#has_key?
and NamedTupleLiteral#has_key?
Thanks for the reviews. Unfortunately, i can't push to |
We'll merge the PR when it gets accepted. For that we still need an approval from another Core Team member. |
Co-authored-by: Quinton Miller <[email protected]>
This PR adds
HashLiteral#has_key?
andNamedTupleLiteral#has_key?
.closes #14841