Skip to content

Drop support for Rails 7.0 and Ruby 3.1 #1045

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

Merged
merged 1 commit into from
Mar 28, 2025
Merged

Drop support for Rails 7.0 and Ruby 3.1 #1045

merged 1 commit into from
Mar 28, 2025

Conversation

sej3506
Copy link
Contributor

@sej3506 sej3506 commented Mar 28, 2025

Drops support for Rails 7.0, see https://endoflife.date/rails
Drops support for Ruby 3.1, see https://endoflife.date/ruby

Also removing HTMLEscapeHelper which was a test utility I
added just to support html escaping strings in tests for Rails 7.0.

Drops support for Rails 7.0, see https://endoflife.date/rails
Drops support for Ruby 3.1, see https://endoflife.date/ruby
- ruby: "3.1.6"
gemfile: "8.0"
- "3.2.8"
- "3.3.7"
Copy link
Contributor

Choose a reason for hiding this comment

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

separate issue, but we need to test on 3.4.2

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sigh, I GUESS 😆

Copy link
Contributor

@MatheusRich MatheusRich left a comment

Choose a reason for hiding this comment

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

LGTM.

@MatheusRich
Copy link
Contributor

@sej3506 are you planning on cutting a new release? Should we add a CHANGELOG entry?

@sej3506
Copy link
Contributor Author

sej3506 commented Mar 28, 2025

@sej3506 are you planning on cutting a new release? Should we add a CHANGELOG entry?

Yes, I plan to do that in a separate commit 👍🏻

@sej3506 sej3506 merged commit fde1586 into main Mar 28, 2025
7 checks passed
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