Skip to content
This repository was archived by the owner on Oct 29, 2025. It is now read-only.

fix: add missing env variables#325

Merged
keithwillcode merged 5 commits intomainfrom
zomars/dops-27-docker-builds-are-broken-because-of-direct_database_url
Feb 8, 2024
Merged

fix: add missing env variables#325
keithwillcode merged 5 commits intomainfrom
zomars/dops-27-docker-builds-are-broken-because-of-direct_database_url

Conversation

@zomars
Copy link
Copy Markdown
Contributor

@zomars zomars commented Feb 7, 2024

Adds missing DIRECT_DATABASE_URL env variables that we're breaking docker builds.

@linear
Copy link
Copy Markdown

linear bot commented Feb 7, 2024

@zomars zomars marked this pull request as ready for review February 7, 2024 16:34
@zomars zomars requested a review from keithwillcode February 7, 2024 16:34
Copy link
Copy Markdown
Contributor

@krumware krumware left a comment

Choose a reason for hiding this comment

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

Does this variable need to be added to the table in the readme?

Outside of that, looks good to me, and looks like it will be backwards compatible so that users don't have to update their .env files out of the gate.

Copy link
Copy Markdown
Contributor

@krumware krumware left a comment

Choose a reason for hiding this comment

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

Thanks for the readme update @keithwillcode. LGTM

@keithwillcode keithwillcode merged commit 91015a7 into main Feb 8, 2024
@keithwillcode keithwillcode deleted the zomars/dops-27-docker-builds-are-broken-because-of-direct_database_url branch February 8, 2024 13:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants