-
Notifications
You must be signed in to change notification settings - Fork 175
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
Use ruby-3.2 for redmine-5.1 #309
Use ruby-3.2 for redmine-5.1 #309
Conversation
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.
Looks good, thank you! Just one minor whitespace nit and can you squash your commits?
(I'm happy to apply these changes for you if you'd prefer 🙇 ❤️)
versions.sh
Outdated
declare -A rubyVersions=( | ||
[5.0]='3.1' |
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.
Minor whitespace nit:
[5.0]='3.1' | |
[5.0]='3.1' |
(Should be tabs to match the rest of the file.)
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.
@tianon Thank you for your response! ✨ 🙇
I ran following (checks no other changes 👀 ), committed it, edited 7d1d1f1 by git rebase -i
and ran force push.
yuya@yoshiyuki|18:56:42|0|12.4% git diff
diff --git a/versions.sh b/versions.sh
index 82b001a..39d9cb2 100755
--- a/versions.sh
+++ b/versions.sh
@@ -19,7 +19,7 @@ declare -A alpineVersions=(
# see https://www.redmine.org/projects/redmine/wiki/redmineinstall
defaultRubyVersion='3.2'
declare -A rubyVersions=(
- [5.0]='3.1'
+ [5.0]='3.1'
)
cd "$(dirname "$(readlink -f "$BASH_SOURCE")")"
yuya@yoshiyuki|18:56:44|0|12.4% git add -u
yuya@yoshiyuki|18:56:49|0|12.4% ./update.sh
5.0: 5.0.7
5.1: 5.1.1
processing 5.0/bookworm ...
processing 5.0/alpine3.18 ...
processing 5.1/bookworm ...
processing 5.1/alpine3.18 ...
yuya@yoshiyuki|18:57:09|0|12.4% git diff | wc -l
0
19a7612
to
dd3e3aa
Compare
Thank you! |
Changes: - docker-library/redmine@5b28bca: Use ruby-3.2 for redmine-5.1 (docker-library/redmine#309)
Changes: - docker-library/redmine@5b28bca: Use ruby-3.2 for redmine-5.1 (docker-library/redmine#309)
Redmine 5.1 can use Ruby 3.2 🎉: https://www.redmine.org/projects/redmine/wiki/redmineinstall
This pull-request makes us using redmine-5.1 on ruby-3.2.
7d1d1f158b7f5e .19a7612dd3e3aa ) runs./update.sh
.