Skip to content

Standardrb#6241

Draft
tvdeyen wants to merge 2 commits intosolidusio:mainfrom
blish:standardrb
Draft

Standardrb#6241
tvdeyen wants to merge 2 commits intosolidusio:mainfrom
blish:standardrb

Conversation

@tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented May 28, 2025

Summary

  • Manually fix remaining offences
  • Add standard-rails?

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.

@github-actions github-actions bot added changelog:solidus_api Changes to the solidus_api gem changelog:solidus_backend Changes to the solidus_backend gem changelog:solidus_core Changes to the solidus_core gem changelog:solidus_sample Changes to the solidus_sample gem changelog:solidus Changes to the solidus meta-gem changelog:solidus_admin changelog:solidus_legacy_promotions Changes to the solidus_legacy_promotions gem changelog:solidus_promotions Changes to the solidus_promotions gem labels May 28, 2025
@tvdeyen tvdeyen mentioned this pull request May 28, 2025
4 tasks
@chaimann
Copy link
Contributor

huge! not sure if you've finished fixing all the offences, but if not, we could potentially benefit from generating a todo with all the remaining offences and gradually release each fix. this way we'll already have a configuration in place and be able to adopt it in new pull requests

Using standardrb --fix-unsafely and manualy correct
where the fix was not working.
@codecov
Copy link

codecov bot commented Sep 11, 2025

Codecov Report

❌ Patch coverage is 94.62366% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.06%. Comparing base (839752f) to head (c5484db).

Files with missing lines Patch % Lines
...erators/solidus_admin/install/install_generator.rb 0.00% 4 Missing ⚠️
.../controllers/solidus_admin/addresses_controller.rb 57.14% 3 Missing ⚠️
...app/controllers/solidus_admin/orders_controller.rb 75.00% 2 Missing ⚠️
...omponents/solidus_admin/products/show/component.rb 0.00% 1 Missing ⚠️
...onents/solidus_admin/stock_items/edit/component.rb 0.00% 1 Missing ⚠️
...n/ui/forms/address/fieldsets/location/component.rb 0.00% 1 Missing ⚠️
...mponents/solidus_admin/ui/pages/index/component.rb 94.44% 1 Missing ⚠️
...lidus_admin/users/store_credits/index/component.rb 66.66% 1 Missing ⚠️
.../controllers/solidus_admin/customers_controller.rb 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6241      +/-   ##
==========================================
- Coverage   89.35%   88.06%   -1.30%     
==========================================
  Files         961      758     -203     
  Lines       20195    15284    -4911     
==========================================
- Hits        18046    13460    -4586     
+ Misses       2149     1824     -325     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@AlistairNorman AlistairNorman mentioned this pull request Feb 26, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:solidus_admin changelog:solidus_api Changes to the solidus_api gem changelog:solidus_backend Changes to the solidus_backend gem changelog:solidus_core Changes to the solidus_core gem changelog:solidus_legacy_promotions Changes to the solidus_legacy_promotions gem changelog:solidus_promotions Changes to the solidus_promotions gem changelog:solidus_sample Changes to the solidus_sample gem changelog:solidus Changes to the solidus meta-gem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants