-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add documentation on S3 scratch bucket #69
Conversation
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.
I added 3 suggestions, but everything else looks great! Thanks Scott!
Not sure if this is a requirement here, but should the notebook be committed as rendered? More of a curious question. |
Also along the lines of 'rendered version', would it be good to have this example not tailored to your GitHub username and use a generic one? |
CryoCloudWebsite/book/_config.yml Lines 36 to 37 in 79d39c9
Based on that setting the notebook must be rendered. Probably worth opening another issue to discuss simplifying github actions borrowed from the uw hackweeks. |
Good point! I'll use |
Addresses #68