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

Rails 2, 3, and 4 LTS tests #1115

Merged
merged 5 commits into from
Jan 28, 2021
Merged

Rails 2, 3, and 4 LTS tests #1115

merged 5 commits into from
Jan 28, 2021

Conversation

schneems
Copy link
Contributor

@schneems schneems commented Jan 27, 2021

Rails version support is limited by a few things:

  • Rails library support by maintainers
  • Ruby version support
  • Behavior support in the Ruby buildpack

Ruby version support

We only provide the latest non-EOL versions of Rubies on stacks. That means when cedar-14 went away, older versions of Ruby such as Ruby 1.9 are no longer available on the platform.

Rails library support by maintainers

We do not patch Rails versions. This includes not patching for security, and not patching for Ruby version compatibility. A customer's ability to run any given Rails version on the platform is limited by the version of Ruby that it can run on.

RailsLTS backports Ruby version compatibility to Rails 2, 3, and 4.

Behavior support in the Ruby buildpack

We will continue providing support for the behavior provided by the buildpack to Rails 2,3, and 4 customers provided. However we hit an issue where we could not run tests for these behaviors because the versions on Rubygems do not work with versions of Ruby available on current stacks.

To address this we're working with the RailsLTS team who have given us a license to use for our test suite. Now these versions of Rails can continue to be supported by Heroku maintainers along with tests for that behavior.

@schneems schneems requested a review from a team as a code owner January 27, 2021 20:38
The current Rails 3 LTS release does not work with Heroku Postgres and it makes a connection to the DB on boot, so I've deleted the Rakefile. Asserting behavior against the rest of the app is better than nothing. If it gets fixed from makandra later it can be re-visited.
Copy link
Member

@edmorley edmorley left a comment

Choose a reason for hiding this comment

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

🎉

@schneems schneems merged commit 1f59972 into main Jan 28, 2021
@schneems schneems deleted the schneems/rails_lts branch January 28, 2021 16:52
@schneems
Copy link
Contributor Author

Apparently my check changelog action isn't firing :(

krisrang added a commit to skyltmax/heroku-buildpack-ruby that referenced this pull request May 13, 2021
* upstream/main: (52 commits)
  Fix date (heroku#1152)
  Bundler 2.2.16 (heroku#1150)
  v226 (heroku#1149)
  [changelog skip] Remove Heroku-16 from tests: (heroku#1146)
  [close 1135] Update bundler 2.x to 2.2.15 (heroku#1144)
  [changelog skip] Clean up mime magic in repos (heroku#1145)
  Clean up CNB release process (heroku#1139)
  Use correct method name in script heroku#1129 (heroku#1129)
  Post-release updates: heroku/ruby 0.1.0 (heroku#1138)
  Add package.toml (heroku#1137)
  Add CNB release scripts (heroku#1136)
  Release v225 (heroku#1133)
  [close heroku#1126] Update bundler to 2.2.11 (heroku#1132)
  Update buildpack tests (heroku#1131)
  v224 (heroku#1128)
  Use jvm buildpack to install JRuby dependencies (heroku#1119)
  Fix YAML formatting (heroku#1116)
  Rails 2, 3, and 4 LTS tests (heroku#1115)
  v223 (heroku#1114)
  Fail detection with a CNB-friendly exit code (heroku#1112)
  ...
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.

2 participants