-
Notifications
You must be signed in to change notification settings - Fork 186
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
Make psql shell #1627
base: edge
Are you sure you want to change the base?
Make psql shell #1627
Conversation
Makefile
Outdated
@@ -86,14 +86,18 @@ e2e-run-all: ## Run E2E tests: all | |||
$(E2E_RUN) npm run e2e:all | |||
|
|||
|
|||
psql-shell: ## Assuming a system is already running with `make start`, start up an interactive psql shell | |||
docker-compose exec postgres psql --username postgres --dbname polis-dev |
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.
@ballPointPenguin Should this actually be --db-name polis-$TAG
? Not sure how dev
gets set, but assuming that's not right for production connections...
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.
Maybe it should pull DB_NAME from env
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.
it's POSTGRES_DB in env.
@ballPointPenguin Seeing this is approved, is this something we want to merge? |
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.
update DB name according to env
00739b3
to
d010378
Compare
I updated this to use the provided .env to determine how to connect to |
This is a just a little
Makefile
addition for firing up a psql shell by runningmake psql-shell
.