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
Upgrade Ruby from 2.5.3 to 2.7.3 #16349
Conversation
Code Climate has analyzed commit fa4161c and detected 0 issues on this pull request. View more on Code Climate. |
…ment-of-veterans-affairs/caseflow into bgantick/ruby-upgrade-2.7
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.
So excited to see this coming to fruition! ❣️
@@ -85,8 +85,6 @@ group :test, :development, :demo do | |||
gem "brakeman" | |||
gem "bullet" | |||
gem "bundler-audit" | |||
# Call 'byebug' anywhere in the code to stop execution and get a debugger console | |||
gem "byebug", platforms: :ruby |
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.
Just for posterity, I asked about this in Slack and the answer is that pry-byebug
gives us byebug, so this isn't actually removing the functionality.
@@ -4,7 +4,7 @@ | |||
factory :intake do | |||
veteran_file_number { create(:veteran).file_number } | |||
detail { create(:higher_level_review) } | |||
type { detail.class.name << "Intake" } | |||
type { detail.class.name + "Intake" } |
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.
Whoa, I wouldn't have guessed <<
was ever allowed for string concatenation. 👍
check judge_task_one.id.to_s, allow_label_click: true | ||
check judge_task_two.id.to_s, allow_label_click: true | ||
page.find(:css, "input[name='#{judge_task_one.id}']", visible: false).execute_script("this.click()") | ||
page.find(:css, "input[name='#{judge_task_two.id}']", visible: false).execute_script("this.click()") |
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.
It is weird to me that this is the only test that needed any updates. But I guess that's a good thing! :)
Description
Upgrades Caseflow's Ruby version from 2.5.3 to 2.7.3. Some additional gem updates for compatibility.
Acceptance Criteria
Testing Plan