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
shared data and username emphasis #103
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.
Formatting changes + add sudo everywhere.
docs/howto/share-data.rst
Outdated
#. Create a terminal session within your JupyterHub interface. | ||
#. Create a folder where your data will live: | ||
|
||
``` |
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.
This is markdown, no?
docs/howto/share-data.rst
Outdated
mkdir /srv/data/mydatafolder | ||
``` | ||
#. Download the data into this folder. For example, using `curl` or by running | ||
a `python` script that downloads the data. |
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.
Also markdown.
docs/howto/share-data.rst
Outdated
``` | ||
#. Create a symbolic link to the data folder | ||
|
||
``` |
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.
more markdown :)
docs/howto/share-data.rst
Outdated
#. Create a symbolic link to the data folder | ||
|
||
``` | ||
ln -s /src/data/mydatafolder mydatafolder |
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.
needs sudo
docs/howto/share-data.rst
Outdated
#. Create a folder where your data will live: | ||
|
||
``` | ||
mkdir /srv/data/mydatafolder |
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.
needs sudo
docs/howto/share-data.rst
Outdated
``` | ||
mkdir /srv/data/mydatafolder | ||
``` | ||
#. Download the data into this folder. For example, using `curl` or by running |
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.
They need to use sudo for this too.
@choldgraf when it is changes like this, do I have a general ok from you to just make the fixes and merge? I know we have it for other projects, but wanted to explicitly check :) |
- Add sudo wherever required
@choldgraf I've implemented all the changes and am gonna merge soon. LMK if you don't want me to do this in the future :) |
@yuvipanda sorry I missed your last comment, yes you definitely have permission from me to do this! |
(if I disagree, I'll just tell you I disagree and/or open a PR :-) ) |
This is some wording to show people how to distribute data with their users. Let me know what you think!
closes #101
closes #102 (by a rebase accident? idk!)