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

Bump latest Ruby versions #422

Merged
merged 3 commits into from
Dec 25, 2017
Merged

Conversation

stanhu
Copy link
Contributor

@stanhu stanhu commented Dec 25, 2017

@@ -46,7 +46,7 @@ def warn_syntax_deviation(feature, version)
CurrentRuby = Ruby23

when /^2\.4\./
current_version = '2.4.0'
current_version = '2.4.3'
Copy link
Owner

Choose a reason for hiding this comment

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

Nope, we still don't even support 2.4.1. If you remove this change I'll merge the rest.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Interesting, thanks!

@whitequark whitequark merged commit 6454edc into whitequark:master Dec 25, 2017
@stanhu
Copy link
Contributor Author

stanhu commented Dec 25, 2017

Thanks! Would you mind releasing a new version?

@ghost
Copy link

ghost commented Jan 8, 2018

@whitequark Do you have a release planned including these changes?

@whitequark
Copy link
Owner

When I have time for it, yes.

rspeicher pushed a commit to gitlabhq/gitlabhq that referenced this pull request Mar 14, 2018
These warnings would show when running rspec:

```
warning: parser/current is loading parser/ruby23, which recognizes
warning: 2.3.5-compliant syntax, but you are running 2.3.6.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
```

Related PRs:

* whitequark/parser#422
* mbj/unparser#79
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.

2 participants