Skip to content
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

[WIP] Add Dockerfile docker-compose file and bootstrap script #245

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jimmykarily
Copy link
Member

This PR is meant to make development of this project easier by setting up all needed components using docker and docker-compose. @vasikoro maybe you want to give it a try and if this works for you we can merge this.

…r script

This helper script can be used to run commands inside the container
which runs the rails server.

Example usage:
bin/drun bin/rails c          # Starts the rails console inside the container
bin/drun bin/rake db:migrate  # Runs migrations
bin/drun bash                 # Starts a bash shell inside the container
@jimmykarily jimmykarily force-pushed the setup_development_environment branch 8 times, most recently from 3e077d7 to 9c84927 Compare April 22, 2017 10:33
@jimmykarily jimmykarily force-pushed the setup_development_environment branch from 9c84927 to 565bea6 Compare April 22, 2017 10:35
gem install bundler
bundle check || bundle install --deployment --path /vendor/bundle --jobs 2 --retry 2
rake db:create
rake db:reset
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this mean development data loss upon each container start?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is true. Maybe we should skip the database setup and let the developer/user do this manually. What do you think?

@jimmykarily
Copy link
Member Author

I'm still trying to fix the failing tests. They are not relevant to this branch but I will wait until I get them green before we merge anything.

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.

2 participants