-
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
[MNT] Deprecate non-federated query tool config, bump Docker Compose required ver. #217
Conversation
✅ Deploy Preview for neurobagel-documentation ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
- older versions don't support environment variable secrets
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.
one comment -> 🧑🍳
Co-authored-by: Sebastian Urchs <[email protected]>
…robagel/documentation into deprecate-local-query-config
Hey @surchs, just to wrap up a small stale issue into this PR, I added a commit c92a4ab to address Barbara's request for more clarity concerning Let me know if you think the addition is clear enough that |
Yep, that is a good way to address it I think. Thanks for that! |
local_nodes.json
#204Changes proposed in this pull request:
Checklist
Please leave checkboxes empty for PR reviewers
[ENH]
,[FIX]
,[REF]
,[TST]
,[CI]
,[MNT]
,[INF]
) see our Contributing Guidelines for more info)Closes #XXXX