Skip to content

v3: Update dependencies and test against Ruby 3.4 #8614

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
Jan 28, 2025

Conversation

tagliala
Copy link
Contributor

@tagliala tagliala commented Jan 26, 2025

Additionally, in order to allow CI to run against all supported
Ruby versions:

  • Lock net-imap to ~> 0.4.18 on Rails 6.1
  • Lock nokogiri to ~> 1.17.2 on Rails 6.1 and Rails 7.0
  • Lock securerandom to ~> 0.3.2 on Rails 6.1 and Rails 7.0
  • Do not test Rails 7.1 against Ruby 3.0, change it to the oldest
    supported Rails version, 6.1

@tagliala tagliala changed the title Update dependencies v3: Update dependencies Jan 26, 2025
@tagliala tagliala marked this pull request as draft January 26, 2025 20:31
@tagliala tagliala force-pushed the chore/3-0-update-deps branch from a9d95c5 to 0d58021 Compare January 26, 2025 20:35
@tagliala tagliala changed the title v3: Update dependencies v3: Update dependencies and test against Ruby 3.4 Jan 26, 2025
@tagliala tagliala force-pushed the chore/3-0-update-deps branch 2 times, most recently from b6e7053 to 05c04b0 Compare January 26, 2025 20:50
@tagliala tagliala force-pushed the chore/3-0-update-deps branch 3 times, most recently from 65815c6 to bfb142c Compare January 27, 2025 17:05
Additionally, in order to allow CI to run against all supported
Ruby versions:
  - Lock net-imap to ~> 0.4.18 on Rails 6.1
  - Lock nokogiri to ~> 1.17.2 on Rails 6.1 and Rails 7.0
  - Lock securerandom to ~> 0.3.2 on Rails 6.1 and Rails 7.0
  - Do not test Rails 7.1 against Ruby 3.0, change it to the oldest
    supported Rails version, 6.1
@tagliala tagliala force-pushed the chore/3-0-update-deps branch from bfb142c to 08f7867 Compare January 27, 2025 17:05
@tagliala tagliala marked this pull request as ready for review January 27, 2025 17:10
Copy link

codecov bot commented Jan 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (3-0-stable@8045939). Learn more about missing BASE report.

Additional details and impacted files
@@              Coverage Diff              @@
##             3-0-stable    #8614   +/-   ##
=============================================
  Coverage              ?   99.19%           
=============================================
  Files                 ?      194           
  Lines                 ?     4968           
  Branches              ?        0           
=============================================
  Hits                  ?     4928           
  Misses                ?       40           
  Partials              ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -33,11 +34,17 @@ jobs:
include:
- ruby: '3.0'
os: ubuntu-latest
deps: rails_71
deps: rails_61
Copy link
Contributor Author

Choose a reason for hiding this comment

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

switched to the oldest supported version (3.0 + 6.1)

@@ -430,4 +436,4 @@ DEPENDENCIES
zeitwerk (~> 2.6.18)

BUNDLED WITH
2.6.2
2.5.23
Copy link
Contributor Author

Choose a reason for hiding this comment

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

2.6 is not available on Ruby 3.0, this downgrade removes a warning on the CI

@tagliala tagliala requested a review from javierjulio January 27, 2025 17:15
Copy link
Member

@javierjulio javierjulio left a comment

Choose a reason for hiding this comment

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

Thanks!

@tagliala tagliala merged commit 73b88e1 into 3-0-stable Jan 28, 2025
24 checks passed
@tagliala tagliala deleted the chore/3-0-update-deps branch January 28, 2025 18:24
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