Skip to content

Remove support for Ruby v2.7#259

Merged
waiting-for-dev merged 1 commit into
masterfrom
waiting-for-dev/deprecate_ruby_v3
Mar 30, 2023
Merged

Remove support for Ruby v2.7#259
waiting-for-dev merged 1 commit into
masterfrom
waiting-for-dev/deprecate_ruby_v3

Conversation

@waiting-for-dev

Copy link
Copy Markdown
Contributor

Summary

Ruby v2.7 will reach EOL on 2023-03-31, so we'll be removing support on Solidus very soon.

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

The following are not always needed:

  • 📖 I have updated the README to account for my changes.
  • 📑 I have documented new code with YARD.
  • 🛣️ I have opened a PR to update the guides.
  • ✅ I have added automated tests to cover my changes.
  • 📸 I have attached screenshots to demo visual changes.

@elia elia 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.

@waiting-for-dev waiting-for-dev force-pushed the waiting-for-dev/deprecate_ruby_v3 branch 3 times, most recently from bab9496 to 3545184 Compare March 30, 2023 03:54
Ruby v2.7 will reach EOL on 2023-03-31, so we'll be removing support on
Solidus very soon.
@waiting-for-dev waiting-for-dev force-pushed the waiting-for-dev/deprecate_ruby_v3 branch from 3545184 to 6ab215e Compare March 30, 2023 04:09
@waiting-for-dev

Copy link
Copy Markdown
Contributor Author

I think we need to correct the ruby version for the linter https://github.com/solidusio/solidus_stripe/blob/waiting-for-dev/deprecate_ruby_v3/.circleci/config.yml#L86

Good call! For now, I have parametrized the parameter on our end to avoid seeing failure in other extensions

@waiting-for-dev waiting-for-dev requested a review from elia March 30, 2023 04:15
@codecov

codecov Bot commented Mar 30, 2023

Copy link
Copy Markdown

Codecov Report

Merging #259 (6ab215e) into master (682579e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #259   +/-   ##
=======================================
  Coverage   99.79%   99.79%           
=======================================
  Files          25       25           
  Lines         481      481           
=======================================
  Hits          480      480           
  Misses          1        1           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@rainerdema rainerdema left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@waiting-for-dev waiting-for-dev merged commit 7396336 into master Mar 30, 2023
@waiting-for-dev waiting-for-dev deleted the waiting-for-dev/deprecate_ruby_v3 branch March 30, 2023 08:51
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.

4 participants