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

DEV: Correct the ruby version check #12551

Merged
merged 2 commits into from
Mar 30, 2021
Merged

DEV: Correct the ruby version check #12551

merged 2 commits into from
Mar 30, 2021

Conversation

tisonkun
Copy link
Contributor

also fix discourse cannot work with ruby 3.0.0

also fix discourse cannot work with ruby 3.0.0
@CLAassistant
Copy link

CLAassistant commented Mar 29, 2021

CLA assistant check
All committers have signed the CLA.

@tisonkun tisonkun changed the title compare semver elegantly compare semver elegantly & compatible with ruby 3.0.0 Mar 29, 2021
@tisonkun
Copy link
Contributor Author

for totally compatible with ruby 3.0.0 we still need to fix issues such as this one.

but change the way we compare semver is always a net win.

config/application.rb Outdated Show resolved Hide resolved
@CvX CvX changed the title compare semver elegantly & compatible with ruby 3.0.0 DEV: Correct the ruby version check Mar 30, 2021
@CvX CvX merged commit 21b4fef into discourse:master Mar 30, 2021
@CvX
Copy link
Contributor

CvX commented Mar 30, 2021

Thank you! 😃

@tisonkun
Copy link
Contributor Author

@CvX thanks for your review! I'll try to run discourse on Ruby 3.0.0 later :-)

@tisonkun tisonkun deleted the patch-1 branch March 30, 2021 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants