Skip to content

Conversation

@shoootyou
Copy link
Contributor

Added documentation for WSL.

Reference: MicrosoftDocs/azure-docs#83871

Fix documentation for WSL
@shoootyou
Copy link
Contributor Author

@microsoft-github-policy-service agree

@prmerger-automator
Copy link
Contributor

@shoootyou : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit de2671f:

✅ Validation status: passed

File Status Preview URL Details
includes/functions-install-core-tools.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Court72
Copy link
Contributor

Court72 commented Feb 5, 2024

@ggailey777

Can you review the proposed changes?

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Feb 5, 2024
@American-Dipper
Copy link
Contributor

sent email and Teams msg to author

@AnnaMHuff
Copy link
Contributor

PR review team emailed author today

@v-ccolin
Copy link
Contributor

I sent an email to the content owner on 01/12/2025.

@MicrosoftDocs/public-repo-pr-review-team

@v-dirichards
Copy link
Contributor

I sent an email to the content owner today.

@MicrosoftDocs/public-repo-pr-review-team

@v-dirichards
Copy link
Contributor

@ggailey777

Can you review this old PR and determine whether it needs to be closed or merged?

@MicrosoftDocs/public-repo-pr-review-team

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds WSL-specific installation documentation for Azure Functions Core Tools. This provides dedicated instructions for installing Core Tools on Windows Subsystem for Linux (WSL) Ubuntu/Debian distributions.

Key Changes

  • Added a new WSL tab section with complete installation instructions
  • Provided WSL-specific APT repository configuration commands
  • Included standard package installation steps adapted for WSL environment

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

##### WSL

```bash
sudo sh -c 'echo "deb [arch=amd64] https://packages.microsoft.com/repos/microsoft-ubuntu-focal-prod focal main" > /etc/apt/sources.list.d/dotnetdev.list'
Copy link

Copilot AI Sep 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The repository URL hardcodes 'focal' (Ubuntu 20.04), which may not be appropriate for all WSL Ubuntu/Debian distributions. Consider using a more generic approach or documenting the version-specific nature of this command.

Suggested change
sudo sh -c 'echo "deb [arch=amd64] https://packages.microsoft.com/repos/microsoft-ubuntu-focal-prod focal main" > /etc/apt/sources.list.d/dotnetdev.list'
# Replace <distro-codename> with the codename for your distribution (see table above, e.g., "focal" for Ubuntu 20.04, "jammy" for Ubuntu 22.04, "bullseye" for Debian 11, etc.)
sudo sh -c 'echo "deb [arch=amd64] https://packages.microsoft.com/repos/microsoft-ubuntu-<distro-codename>-prod <distro-codename> main" > /etc/apt/sources.list.d/dotnetdev.list'

Copilot uses AI. Check for mistakes.

1. Set up the APT source list before doing an APT update.

##### WSL
Copy link

Copilot AI Sep 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] The heading '##### WSL' appears redundant since this entire section is already under the WSL tab. This subheading doesn't add clarity and could be removed.

Suggested change
##### WSL

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants