search

carpentries-incubator / cwl-novice-tutorial

Introduction to Workflows with Common Workflow Language
https://carpentries-incubator.github.io/cwl-novice-tutorial/
Other
11 stars 20 forks source link

Separate and expand setup instructions #82

Closed gcapes closed 2 years ago

gcapes commented 2 years ago

Fix #80.

mr-c commented 2 years ago

Do you want to grab my changes in #81 or should I try to push commits here?

netlify[bot] commented 2 years ago

✔️ Add Deploy Preview notifications as pull request comments when Deploy Preview succeeds

🔨 Explore the source changes: 22c4c153777c0076aa0a8c026bcb2231816e9706

🔍 Inspect the deploy log: https://app.netlify.com/sites/thirsty-hoover-1e0704/deploys/62332294b9ac990008d161a8

😎 Browse the preview: https://deploy-preview-82--thirsty-hoover-1e0704.netlify.app

gcapes commented 2 years ago

@mr-c I've copied one of the changes, but I'm unclear on the rest.

Is the suggestion that Windows users install VSCode natively, then use an extension to access files in WSL2? It's not super clear whether that's the suggestion, and if so would it be simpler to install VS code within WSL2? Similar for accessing git through VSCode instead of a shell.

It would make teaching simpler if learners on different OS are using as similar a set up to each other as possible.

Would you like to make the relevant changes?

mr-c commented 2 years ago

if so would it be simpler to install VS code within WSL2?

Good question. If you install VS Code within WSL2, one gets this message

To use Visual Studio Code with the Windows Subsystem for Linux, please install Visual Studio Code in Windows and uninstall the Linux version in WSL. You can then use the code command in a WSL terminal just as you would in a normal command prompt.

So the instructions are to install VS Code in Windows. I'll push some commits to bring in the rest of my changes.