Skip to content

Drop support for Rails 5.2 & Ruby 2.7#58

Merged
smudge merged 3 commits into
mainfrom
drop_older_rails_support
Sep 12, 2025
Merged

Drop support for Rails 5.2 & Ruby 2.7#58
smudge merged 3 commits into
mainfrom
drop_older_rails_support

Conversation

@Irving-Betterment

@Irving-Betterment Irving-Betterment commented Sep 5, 2025

Copy link
Copy Markdown
Contributor

Summary of changes:
This pull request drops support for the oldest Rails and Ruby versions, in order to streamline the supportability bracket. By doing so, It also cuts a minor version release.

@Irving-Betterment
Irving-Betterment force-pushed the drop_older_rails_support branch 2 times, most recently from 8cf72fe to 1cbc5ab Compare September 5, 2025 15:32
@Irving-Betterment Irving-Betterment changed the title Drop support for < Rails 7.2 & < Ruby 3.1 Drop support for Rails 5.2 Sep 5, 2025
@Irving-Betterment
Irving-Betterment marked this pull request as ready for review September 5, 2025 15:35
@Irving-Betterment
Irving-Betterment marked this pull request as draft September 5, 2025 16:13
@Irving-Betterment
Irving-Betterment marked this pull request as ready for review September 5, 2025 17:16

@argvniyx-enroute argvniyx-enroute left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

domainlgtm

@Irving-Betterment Irving-Betterment changed the title Drop support for Rails 5.2 Drop support for Rails 5.2 & Ruby 2.7 Sep 8, 2025
Comment thread lib/delayed/version.rb
# frozen_string_literal: true

module Delayed
VERSION = '1.0.0'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think we still stick to a minor bump when dropping a ruby version. See this webvalve commit, for example. Its parent is a commit dropping older ruby and rails versions.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I second on being the preferred approach (so far). However, this internal conversation brought the necessity for a major version cut. cc @smudge

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Ah, I see, I wasn't aware of that convo.

@Irving-Betterment Irving-Betterment Sep 12, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Follow-up: The updated consensus is to keep cutting minors for dependency drops.

@argvniyx-enroute argvniyx-enroute left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

domainlgtm

smudge
smudge previously approved these changes Sep 10, 2025

@smudge smudge left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

domain LGTM && platform LGTM

@Irving-Betterment

Copy link
Copy Markdown
Contributor Author

@smudge can you please help me merging this approved PR?

@argvniyx-enroute argvniyx-enroute left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

domainlgtm

@smudge smudge left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

domain LGTM && platform LGTM

@smudge
smudge merged commit d46a0fb into main Sep 12, 2025
47 checks passed
@smudge
smudge deleted the drop_older_rails_support branch September 12, 2025 21:12
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.

3 participants