Skip to content
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

[Ruby 3.0] Reject ->... and ->(...) using the same error #4367

Merged
merged 2 commits into from Jul 15, 2021

Conversation

vinistock
Copy link
Collaborator

Implement this change: whitequark/parser#713

Based on my reviews of the commits in whitequark, this should be the last of the parser/lexer/builder changes for feature parity with Ruby 3.0 (unless I missed something 😅).

This PR updates the import whitequark script with the SHA of the latest Ruby 3.0 parser commit and sets the target as Ruby 3.0.

Additionally, it implements the brief lexer change mentioned.

Motivation

Complete Ruby 3.0 support.

Test plan

The existing tests that were previously imported should cover this.

@vinistock vinistock requested a review from a team as a code owner July 15, 2021 13:52
@vinistock vinistock requested review from aprocter-stripe and removed request for a team July 15, 2021 13:52
@aprocter-stripe
Copy link
Contributor

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR. When the build
finishes, I'll share with you whether or how it failed. Thanks!

Stripe employees can see the build results here:

https://go/builds/bui_JrGwbsXf0YtDak
https://go/builds/bui_JrGwpmcYDCFR03

Copy link
Contributor

@aprocter-stripe aprocter-stripe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@aprocter-stripe aprocter-stripe merged commit 5df4c8b into sorbet:master Jul 15, 2021
@vinistock vinistock deleted the ruby30-parser-updates branch July 15, 2021 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants