-
Notifications
You must be signed in to change notification settings - Fork 685
docs: Document Azure environment variables #6084
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
Conversation
The Azure environment variables were not documented. This PR adds them to the documentation pages. Signed-off-by: adamrtalbot <[email protected]>
✅ Deploy Preview for nextflow-docs-staging ready!
To edit notification comments on pull requests, go to your Netlify project 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.
Very minor language suggestions.
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Adam Talbot <[email protected]>
Note this one is superseded by #6087 |
Co-authored-by: Adam Talbot <[email protected]> Signed-off-by: Paolo Di Tommaso <[email protected]>
@adamrtalbot should this closed or merged? |
Merge so we have the envs documented right now. If #6087 gets merged it includes the content from this PR anyway. |
The writing itself looks good, but a second set of approvals from someone who understands Azure more than me should check as well. |
Signed-off-by: Paolo Di Tommaso <[email protected]>
Nice |
The Azure environment variables were not documented. This PR adds them to the documentation pages.