Skip to content

Update Gemfile for new spree gem location#12

Merged
damianlegawiec merged 1 commit intomainfrom
fix/spree-gem-path
Feb 11, 2026
Merged

Update Gemfile for new spree gem location#12
damianlegawiec merged 1 commit intomainfrom
fix/spree-gem-path

Conversation

@damianlegawiec
Copy link
Copy Markdown
Member

@damianlegawiec damianlegawiec commented Feb 11, 2026

Summary

  • Updates the Bundler github source for spree/spree to include glob: 'backend/engines/**/*.gemspec'
  • The spree gems have been moved from the root of the spree/spree repository to backend/engines/

Test plan

  • bundle update succeeds
  • bundle exec rake test_app succeeds
  • bundle exec rspec passes (26 examples, 0 failures)

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated gem dependency resolution configuration to support additional package sources.

The spree gems have been moved from the root of the spree/spree repository
to backend/engines/. This adds the glob parameter to the github source
so Bundler can locate the gemspecs in their new path.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 11, 2026

Walkthrough

The Gemfile is updated to include a glob pattern for backend engine gemspecs when SPREE_PATH is not set, extending gem source resolution to cover additional gemspec files in the backend/engines directory.

Changes

Cohort / File(s) Summary
Gemfile Configuration
Gemfile
Added glob pattern to spreeOpts hash for backend/engines/**/*.gemspec to expand gem source resolution when SPREE_PATH is not configured.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A glob so small, a pattern neat,
Backend engines, now complete!
Gemspecs dance in organized rows,
Where the SPREE_PATH doesn't go.
One line added, clarity grows! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating the Gemfile to accommodate the new location of spree gems in the backend/engines directory.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/spree-gem-path

No actionable comments were generated in the recent review. 🎉

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 RuboCop (1.84.1)
Gemfile

Error: The Metrics/LineLength cop has been moved to Layout/LineLength.
(obsolete configuration found in .rubocop.yml, please update it)


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@damianlegawiec damianlegawiec merged commit bae3153 into main Feb 11, 2026
4 checks passed
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.

1 participant