-
-
Notifications
You must be signed in to change notification settings - Fork 127
feat(web): merge postgress instances #1049
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
26107b5 to
5bb166a
Compare
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## develop #1049 +/- ##
========================================
Coverage 12.05% 12.05%
========================================
Files 301 301
Lines 11188 11188
Branches 1176 1176
========================================
Hits 1349 1349
Misses 9814 9814
Partials 25 25 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
394949f to
c20adf1
Compare
nandor-magyar
left a comment
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.
Nice and clean, only one little thing with script.
We had two PostgreSQL instances instead of one. This PR merges them into a single instance and refactors the
docker-compose.yamlfiles accordingly.I skipped the
clirefactor because, after discussing with the internal team, we decided to usedocker composeinstead of theCLI.