Skip to content

Conversation

@HMS17
Copy link
Contributor

@HMS17 HMS17 commented Jul 28, 2025

BI-2700 Improve Technical Documentation For DeltaBreed Paper

Changes to README and additional files to help new users and developers set up a local instance of DeltaBreed.

Copy link
Contributor

@mlm483 mlm483 left a comment

Choose a reason for hiding this comment

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

This looks really good. I left some small suggestions.

One other thing to consider is if there's any point in having two .env templates. If .env.quicktemplate is the best way to get started, why not just replace .env.template with the contents of .env.quicktemplate? Anyone who needs to change a variable with a default value is still free to do so in that case. If there are helpful comments in the .env.template that you want to save, you could move them above their respective variables.

OAUTH_CLIENT_SECRET=<oauth_client_secret goes here>

##Variables already set
JWT_SECRET=insecure2520f59b2bccc64ce155482bc413f6b9ede5c6d418ab529d1b2051baa6f0849e
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks good. 😊

@HMS17 HMS17 marked this pull request as ready for review July 31, 2025 21:09
@HMS17 HMS17 merged commit 3f00936 into develop Jul 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants