Skip to content

CI: Raise Rails 7.2 deprecations#6009

Closed
tvdeyen wants to merge 3 commits intosolidusio:mainfrom
tvdeyen:tackle-rails-7.2-deprecations
Closed

CI: Raise Rails 7.2 deprecations#6009
tvdeyen wants to merge 3 commits intosolidusio:mainfrom
tvdeyen:tackle-rails-7.2-deprecations

Conversation

@tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Dec 5, 2024

In order to tackle deprecations one by one

@tvdeyen tvdeyen self-assigned this Dec 5, 2024
@tvdeyen tvdeyen changed the title CI: Raise Rails deprecations CI: Raise Rails 7.2 deprecations Dec 5, 2024
@github-actions github-actions bot added changelog:repository Changes to the repository not within any gem changelog:solidus_core Changes to the solidus_core gem changelog:solidus_admin and removed changelog:repository Changes to the repository not within any gem labels Dec 5, 2024
@tvdeyen tvdeyen force-pushed the tackle-rails-7.2-deprecations branch from 40ebaac to c333192 Compare December 5, 2024 16:10
In order to tackle deprecations one by one
This is the Rails default.

From the docs

> Eager loading loads your whole application. When running a single test locally,
> this probably isn't necessary. It's a good idea to do in a continuous integration
> system, or in some way before deploying your code.
@tvdeyen tvdeyen force-pushed the tackle-rails-7.2-deprecations branch from c333192 to 3992e39 Compare January 24, 2025 15:33
@codecov
Copy link

codecov bot commented Jan 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.16%. Comparing base (b1b8567) to head (3992e39).
Report is 45 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6009      +/-   ##
==========================================
+ Coverage   89.28%   90.16%   +0.88%     
==========================================
  Files         813      813              
  Lines       17906    17832      -74     
==========================================
+ Hits        15987    16078      +91     
+ Misses       1919     1754     -165     

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

@tvdeyen tvdeyen marked this pull request as ready for review January 24, 2025 16:39
@tvdeyen tvdeyen requested a review from a team as a code owner January 24, 2025 16:39
@fthobe
Copy link
Contributor

fthobe commented Feb 7, 2025

@jarednorman can we merge this?

@tvdeyen
Copy link
Member Author

tvdeyen commented Feb 14, 2025

Not necessary anymore now that we merged #6091

@tvdeyen tvdeyen closed this Feb 14, 2025
@tvdeyen tvdeyen deleted the tackle-rails-7.2-deprecations branch February 25, 2025 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:solidus_core Changes to the solidus_core gem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants