-
Notifications
You must be signed in to change notification settings - Fork 6
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
[ENH] Update instructions for changing passwords and the default deployment profile #228
Conversation
✅ Deploy Preview for neurobagel-documentation ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Hey @alyssadai. Thanks for all the nice changes!
I disagree with the "here's how you change the default profile" bit. Let's not add that. More generally: let's not just tack a change like that onto an unrelated PR so we have a chance to first discuss it in an issue in the future.
Otherwise all good.
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.
🧑🍳
local_node
tofull_stack
recipes#86 and [REF] Create GraphDB password secrets from .txt files instead of env variables recipes#84Changes proposed in this pull request:
full_stack
COMPOSE_PROFILES
var in .envChecklist
Please leave checkboxes empty for PR reviewers
[ENH]
,[FIX]
,[REF]
,[TST]
,[CI]
,[MNT]
,[INF]
) see our Contributing Guidelines for more info)Closes #XXXX