-
-
Notifications
You must be signed in to change notification settings - Fork 22
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 to ruby 3.1 #3646
Update to ruby 3.1 #3646
Conversation
Matching ansible PR: https://github.com/dodona-edu/dodona-ansible/pull/145 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure of I am one 100% fan of the new syntax, but I'll get used to it :)
I agree, I'm more fond of explicit naming, but I guess rubocop has a good reason to do it this way. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I of course agree with bumping this. Can we coordinate this easily or do we need to wait till after the exams (and coordinate this with the yearly server upgrade)
Coordination is quite easy. We can pre-install the new ruby version on our servers while keeping things running with the old version. A passenger restart (which happens as part of the deployment process) then switches to the new version. |
I'm pretty sure the style is configurable. If we prefer writing the keyword/hash args explicitely, we can probably configure it that way. This is just the default. I personally don't have a strong opinion either way. |
I changed the style to disallow the hash shorthand syntax. Didn't like it either. |
b9d5ef8
to
b6bcb92
Compare
No description provided.