-
Notifications
You must be signed in to change notification settings - Fork 111
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to Ruby 3.3.1 #10627
Update to Ruby 3.3.1 #10627
Conversation
45250cf
to
1df408a
Compare
dd657b6
to
c7adbcb
Compare
6772319
to
b8723cc
Compare
# Wait for the the loading interstitial to disappear before continuing | ||
expect(page).not_to have_content(t('doc_auth.headings.interstitial'), wait: 10) | ||
|
||
begin | ||
expect(page).not_to have_content(t('doc_auth.headings.interstitial'), wait: 10) | ||
rescue Selenium::WebDriver::Error::StaleElementReferenceError |
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.
seeing all this repeated, I feel like we could probably build a helper like
wait_for_content_to_disappear do |page|
expect(page).not_to have_content(t('doc_auth.headings.interstitial'), wait: 10)
end
and then that helper method has these rescue
blocks built in
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.
Good call, added in d8d45bf
changelog: Internal, Maintenance, Update to Ruby 3.3.1
Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
b8723cc
to
d8d45bf
Compare
Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
This reverts commit fb74d7b.
馃洜 Summary of changes
Follows #10609 to upgrade CI to run 3.3.1 and updates the repo to use 3.3.1 along with it