Skip to content

Conversation

larouxn
Copy link
Contributor

@larouxn larouxn commented Jul 13, 2025

Proposing we drop support for Rails 7.0 (not 7.1 nor 7.2, yet) as it's no longer supported as of over 3 months ago. Doing so additionally allows us to clean up quite a few if >= 7.1 type conditionals. (Rails EOL info source)

image

@larouxn larouxn force-pushed the drop_rails_7.0_support_and_clean_up branch from ab510ee to 3e8bba4 Compare July 13, 2025 20:29
@larouxn larouxn force-pushed the drop_rails_7.0_support_and_clean_up branch from 3e8bba4 to d60ada4 Compare July 24, 2025 14:23
@larouxn
Copy link
Contributor Author

larouxn commented Jul 24, 2025

Simply rebased on main to ensure a clean merge when ready. Suppose I need another approval to run CI. 🥲

@adrianna-chang-shopify
Copy link
Contributor

Github is stalling on the checks for rails_7_0_gemfile, which was removed from the CI matrix, so I'm just gonna ignore that and merge 😅

@adrianna-chang-shopify adrianna-chang-shopify merged commit c7b7b1b into Shopify:main Jul 25, 2025
14 checks passed
@larouxn larouxn deleted the drop_rails_7.0_support_and_clean_up branch July 25, 2025 22:28
@etiennebarrie
Copy link
Member

Github is stalling on the checks for rails_7_0_gemfile, which was removed from the CI matrix, so I'm just gonna ignore that and merge 😅

FYI the fix is to go into the repository "Branches" settings then edit the branch protection rules for main, then scroll down, remove the statuses we don't want, and not forget to save the form. I've done it, just sharing for next time.

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