-
Notifications
You must be signed in to change notification settings - Fork 142
DOC: Update installation guide in /docs/WSL.md #769
Conversation
- Removed long installation guide - Included new shorter guide to install WSL
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.
Great, thanks very much @arsenkhy for that contribution. Some small updates requested before merging, but I'm approving already.
LGTM once the changes Anton has requested have been added - will run workflows and approve after then :) |
- More clear language
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.
Looks great now! I fixed minor typos.
@arsenkhy you'll need to trim trailing whitespace from the file to pass CI checks (easily done using VS Code ) |
- Trimmed trailing whitespace to pass the CI checks
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@arsenkhy , I had to manually start the PR build, once that has come back as green, feel free to merge! |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Not sure why this is timing out, running again one more time and will investigate more thoroughly if it fails! |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
This will avoid e.g. #769 (comment).
Fixes #766