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.

Sign up for GitHub

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

Jump to bottom

Dockerize #23

Merged
merged 22 commits into from
Nov 21, 2022
Merged

Dockerize #23

merged 22 commits into from
Nov 21, 2022

Conversation

zakariaf
Copy link
Owner

@zakariaf zakariaf commented Nov 13, 2022 •
edited
Loading

Add Docker configs for production

@zakariaf
Use SECRET_KEY_BASE ENV instead of credentials and secrets
0ed6977
@zakariaf
Improve config/puma.rb by considering Active Record and its connectio…
84bb31f 
…n when we are preloading app
@zakariaf
Add postfix to database name for each env
8968ccc
@zakariaf
Add ruby platform for nokogiri gem
b8dbbbe
@zakariaf
Dockerize app
3ed4510
@zakariaf
add -run- file to help interact with dockerized project
3918bb1
@zakariaf
Update bin/vite
60b0abe
@zakariaf
Use alpine version to dockerzie the app
6e4471e
@zakariaf
Enabel Vite hot reloading
b9b2fce
@zakariaf
Add more ENVs
1443bc4
@zakariaf
Make the web container interactive
e782738
@zakariaf
Make public_file_server always true for production
ea29eaf
@zakariaf
Use Docker only for production image
8a50090
@zakariaf
Use development env for Node because of yarn
61ee9dd
@zakariaf
No need for tty as we do not use docker for development stage
393fc63
@zakariaf
NODE_ENV must be always development
915133e
@zakariaf
Keep .env file and remove .env.example
f29d142
@zakariaf
Update README
9d51b33
@zakariaf zakariaf changed the title WIP: Dockerize Dockerize Nov 21, 2022
@zakariaf zakariaf merged commit 341c2f3 into main Nov 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@zakariaf