-
Notifications
You must be signed in to change notification settings - Fork 0
Fix bundler install for docker-container #225
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
Conversation
Reviewer's Guide by SourceryThis pull request fixes the broken bundler configuration for docker-containers. The primary issue was that the separation of development and test environments was not correctly handled during the Updated class diagram for Gemfile dependenciesclassDiagram
class Gemfile {
-source 'https://rubygems.org'
-gem 'activerecord'
-gem 'dalli'
+gem 'debug'
-gem 'erubis'
-gem 'i18n'
-gem 'puma'
-gem 'rack-attack'
-gem 'rackup'
-gem 'rake'
-gem 'require_all'
+gem 'rspec'
-gem 'rufus-scheduler'
-gem 'sinatra'
-gem 'sinatra-activerecord'
-gem 'sprockets'
-gem 'sqlite3'
-group :development
-gem 'brakeman'
-gem 'i18n-tasks'
-gem 'overcommit'
-gem 'rerun'
-gem 'rubocop'
-gem 'rubocop-performance'
-gem 'rubocop-rails'
-group :test
-gem 'database_cleaner'
-gem 'faker'
-gem 'rack-test'
}
note for Gemfile "'debug' and 'rspec' gems moved out of development/test groups"
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @whotwagner - I've reviewed your changes - here's some feedback:
Overall Comments:
- Consider using multi-stage builds to avoid including development and test dependencies in the final image.
- It might be helpful to add a comment explaining why the
without
config is being set locally.
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Bundler install in the docker-container was broken because we seperated development and test environments. This PR fixes the broken bundler configuration for docker-containers
Summary by Sourcery
Fixes the broken bundler configuration for docker-containers by including development and test dependencies during bundle install in the Dockerfile. This ensures that all necessary gems are available in the container environment.
Build: