Skip to content
This repository was archived by the owner on Jan 25, 2023. It is now read-only.

Preinstall Ruby 2.6.2#439

Merged
vbrown608 merged 2 commits intoxenialfrom
438-preinstall-ruby-2.6.2
Jun 26, 2020
Merged

Preinstall Ruby 2.6.2#439
vbrown608 merged 2 commits intoxenialfrom
438-preinstall-ruby-2.6.2

Conversation

@vbrown608
Copy link
Copy Markdown
Contributor

Many users are still pinned to Ruby 2.6.2, and are seeing an error like
ruby
Although we're defaulting to 2.7 for new sites, we should continue to preinstall 2.6.2 or most users builds will be slowed down by about 15 seconds, regardless of whether they're using Ruby.

@vbrown608 vbrown608 requested a review from a team as a code owner June 17, 2020 20:35
Copy link
Copy Markdown
Contributor

@mheffner mheffner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to do the gem bundle install for that version as well?

@vbrown608
Copy link
Copy Markdown
Contributor Author

That's a good point. Buildbot will install Bundler if it's missing (and it installs a lot faster than Ruby) but I think it makes sense to go ahead and preinstall it if we're preinstalling the Ruby version 👍

Copy link
Copy Markdown
Contributor

@mheffner mheffner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍔

@vbrown608 vbrown608 merged commit 3142d77 into xenial Jun 26, 2020
@vbrown608 vbrown608 deleted the 438-preinstall-ruby-2.6.2 branch June 26, 2020 00:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants