Update CONTRIBUTING.md with documentation supporting WSL - #50245
Open
Jair Myree (jairmyree) wants to merge 6 commits into
Open
Update CONTRIBUTING.md with documentation supporting WSL#50245Jair Myree (jairmyree) wants to merge 6 commits into
Jair Myree (jairmyree) wants to merge 6 commits into
Conversation
Added notes for WSL browser linking and Maven authentication.
|
Azure Pipelines: Successfully started running 2 pipeline(s). 35 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
Jair Myree (jairmyree)
requested review from
Alan Zimmer (alzimmermsft) and
vcolin7
August 25, 2026 21:24
Contributor
There was a problem hiding this comment.
Pull request overview
Updates contributor documentation to better support developers using WSL when setting up Maven authentication and related tooling, improving the onboarding experience for cross-platform contributors.
Changes:
- Adds a WSL-specific note about configuring link opening via the host browser (with an Ubuntu
wslusuggestion). - Keeps the existing guidance for Azure DevOps pipeline Maven authentication (MavenAuthenticate@0) in the same section.
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Jesse Squire (jsquire)
approved these changes
Aug 25, 2026
|
|
||
| For detailed instructions, refer to the [Maven Credential Provider documentation](https://eng.ms/docs/coreai/devdiv/one-engineering-system-1es/1es-docs/azure-artifacts/maven-credprovider). | ||
|
|
||
| > **Note:** If you're using WSL, ensure URL opening is configured to use your host OS browser. On Ubuntu-based WSL distros, you can install `wslu` (`sudo apt update && sudo apt install -y wslu`), which provides `wslview`. |
There was a problem hiding this comment.
Is there a reference that we can point to for more information on this? The context is clear if you understand how WSL works but may not be generally understood.
There was a problem hiding this comment.
+1 to Jesse's comment, you could add a link to Windows Subsystem for Linux Documentation
|
|
||
| For detailed instructions, refer to the [Maven Credential Provider documentation](https://eng.ms/docs/coreai/devdiv/one-engineering-system-1es/1es-docs/azure-artifacts/maven-credprovider). | ||
|
|
||
| > **Note:** If you're using WSL, ensure URL opening is configured to use your host OS browser. On Ubuntu-based WSL distros, you can install `wslu` (`sudo apt update && sudo apt install -y wslu`), which provides `wslview`. |
There was a problem hiding this comment.
+1 to Jesse's comment, you could add a link to Windows Subsystem for Linux Documentation
Co-authored-by: Jonathan Cárdenas <JonathanCrd@users.noreply.github.com>
Jair Myree (jairmyree)
requested review from
Jonathan Cárdenas (JonathanCrd) and
Jesse Squire (jsquire)
August 26, 2026 17:46
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added notes for WSL browser linking and Maven authentication.