-
Notifications
You must be signed in to change notification settings - Fork 7
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
workspace Folder #14
Comments
@FlorianPydde Yes that part needs to be updated. You can check this section of README.md regarding how you need to change it. https://github.com/microsoft/dstoolkit-devcontainers#how-to-create-a-new-directory-under-src-with-a-new-environment I just merged new readme.md and new devcontainers version that has no dependency on docker-compose anymore with this #13 so you might have missed the update to README.md there. |
Fair enough. Since I was starting some EDA I was only looking at the notebook folder not the src one. Should we always copy the example folder in src as starting point ? |
If you want to use it only for EDA, you can stick with notebooks directory. If you want to start writing something for more serious experimentation or production code, I'd suggest to use to src. |
Yes, that's what I tried to do, i.e. simply use the notebook folder. But that's when I saw that devcontainer.json "workspaceFolder" key had to be updated, and this is not mentioned in the documentation. |
You don't need to update workspaceFolder key when you use notebooks directory. the latest main got this below
and this should work out of the box. Build succeeded on my local and succeeded on David's local too during his PR review.
|
When notebook or src folder in a new container, should we also change the workspaceFolder in the devcontainer.json config when setting up the project?
Currently it's set to: "workspaceFolder": "/workspaces/dstoolkit-devcontainers)/notebooks"
The text was updated successfully, but these errors were encountered: