Skip to content

Conversation

@github-actions
Copy link

@github-actions github-actions bot commented Dec 3, 2024

Backport

This will backport the following commits from main to v4.2:

Questions ?

Please refer to the Backport tool documentation

We were missing a translation here, but never noticed, because we were
not testing for an actual human-understandable string.

This is broken in the v4.0, v4.1, v4.2 and v4.3 branches with a noisier
error message, because in those branches, `I18n.t("unavailable")` is
different from `t("unavailable")`.

(cherry picked from commit 506ddfc)
@github-actions github-actions bot requested a review from a team as a code owner December 3, 2024 11:25
@github-actions github-actions bot added changelog:solidus_backend Changes to the solidus_backend gem changelog:solidus_core Changes to the solidus_core gem labels Dec 3, 2024
@codecov
Copy link

codecov bot commented Dec 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.86%. Comparing base (cc645f9) to head (a053539).
Report is 2 commits behind head on v4.2.

Additional details and impacted files
@@             Coverage Diff             @@
##             v4.2    #5984       +/-   ##
===========================================
+ Coverage   70.28%   88.86%   +18.57%     
===========================================
  Files         561      607       +46     
  Lines       14304    14750      +446     
===========================================
+ Hits        10053    13107     +3054     
+ Misses       4251     1643     -2608     

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

@tvdeyen tvdeyen merged commit fbd24ae into v4.2 Dec 3, 2024
9 checks passed
@tvdeyen tvdeyen deleted the backport/v4.2/pr-5979 branch December 3, 2024 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:solidus_backend Changes to the solidus_backend gem changelog:solidus_core Changes to the solidus_core gem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants