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

Update Ruby to 3.2.2 for everything and Rails to 6.1.7.6 #174

Merged
merged 12 commits into from
Sep 7, 2023

Conversation

Janell-Huyck
Copy link
Contributor

@Janell-Huyck Janell-Huyck commented Sep 5, 2023

Fixes #168
Upgrade all usages of Ruby to 3.2.2, and all mentions of Rails to 6.1.7.6. Ran bundle update. All tests passing.

The circleci tests would no longer run after updating the Ruby version. Turns out there's an issue with the orb, as described at CircleCI-Public/browser-tools-orb#75. The workaround is to force the Chrome installation to be a specific version (114.0.5735.90) so that the chromeDrivers that the orb can install will work. Applying the workaround also ended up requiring me to specifically add in an installation of Node.js, as it would not work without Node installed. After making these changes, the circleci tests would again run and pass.

Changes to circleci/config.yml file:

  • updated to Ruby 3.2.2
  • changed "browser-tools/install-chrome" to
    "browser-tools/install-browser-tools:
    chrome-version: 114.0.5735.90"
  • added instructions to install Node v 18.17.1

@scherztc scherztc closed this Sep 6, 2023
@scherztc scherztc reopened this Sep 6, 2023
@Janell-Huyck
Copy link
Contributor Author

There is an additional Coveralls check that is trying to run and hanging. Closing this PR and will re-open as two separate PR's, splitting off the CircleCI changes into their own PR.

@Janell-Huyck
Copy link
Contributor Author

Per Glen, there has been a change in the GIthub settings that is causing this to hang and once the earlier commits are merged in this should pass okay. Ok to re-open PR.

@Janell-Huyck Janell-Huyck reopened this Sep 6, 2023
@Janell-Huyck Janell-Huyck reopened this Sep 6, 2023
@scherztc scherztc self-assigned this Sep 7, 2023
@scherztc scherztc merged commit 0deda0b into qa Sep 7, 2023
2 checks passed
@scherztc scherztc deleted the 168-update-ruby-and-rails branch September 7, 2023 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade ruby, rails, and gems
2 participants