-
Notifications
You must be signed in to change notification settings - Fork 1.2k
docs: add note for wsl users about filesystem location. #6101
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
Conversation
fujitatomoya
left a comment
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.
lgtm with green CI.
i would like to have another approval from maitainer.
fujitatomoya
left a comment
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.
@bhumilmayani12-oss there are a couple of typos, https://github.com/ros2/ros2_documentation/actions/runs/20357941200/job/58497149743?pr=6101
can you fix them?
Co-authored-by: Tomoya Fujita <[email protected]> (cherry picked from commit 2e05f75)
Co-authored-by: Tomoya Fujita <[email protected]> (cherry picked from commit 2e05f75)
Co-authored-by: Tomoya Fujita <[email protected]> (cherry picked from commit 2e05f75)
(cherry picked from commit 2e05f75) Co-authored-by: Bhumil Mayani <[email protected]> Co-authored-by: Tomoya Fujita <[email protected]>
(cherry picked from commit 2e05f75) Co-authored-by: Bhumil Mayani <[email protected]> Co-authored-by: Tomoya Fujita <[email protected]>
(cherry picked from commit 2e05f75) Co-authored-by: Bhumil Mayani <[email protected]> Co-authored-by: Tomoya Fujita <[email protected]>
This PR adds a short note for Windows users building the documentation with WSL, recommending working inside the Linux filesystem instead of /mnt/c to avoid common build issues.