Skip to content

Add shebang to indicate bash requirement#12732

Merged
michaeltlombardi merged 2 commits intoMicrosoftDocs:mainfrom
sdwheeler:sdw-w550714-shebang
Feb 4, 2026
Merged

Add shebang to indicate bash requirement#12732
michaeltlombardi merged 2 commits intoMicrosoftDocs:mainfrom
sdwheeler:sdw-w550714-shebang

Conversation

@sdwheeler
Copy link
Collaborator

@sdwheeler sdwheeler commented Feb 4, 2026

PR Summary

Add shebang to indicate bash requirement

PR Checklist

  • Descriptive Title: This PR's title is a synopsis of the changes it proposes.
  • Summary: This PR's summary describes the scope and intent of the change.
  • Contributor's Guide: I have read the contributor's guide.
  • Style: This PR adheres to the style guide.

Copilot AI review requested due to automatic review settings February 4, 2026 15:36
@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

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

This PR adds shebang lines (#!/bin/bash) to shell script examples in Linux installation documentation to explicitly indicate that bash is required. The documentation date fields are also updated to reflect the current date.

Changes:

  • Added #!/bin/bash shebang to shell scripts in Ubuntu, RHEL, Debian, and Alpine installation documentation
  • Updated ms.date fields to 02/04/2026 for all modified files

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
install-ubuntu.md Added bash shebang to 2 script examples (PMC installation and direct download); updated date
install-rhel.md Added bash shebang to PMC installation script; updated date
install-debian.md Added bash shebang to 2 script examples (PMC installation and direct download); updated date
install-alpine.md Added bash shebang to installation script; updated date

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Then, in the terminal, execute the following shell commands to install PowerShell 7.4:

```sh
#!/bin/bash
Copy link

Copilot AI Feb 4, 2026

Choose a reason for hiding this comment

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

The shebang specifies bash, but Alpine Linux does not include bash by default and the script does not use any bash-specific features. Alpine uses ash (Almquist shell) by default. Consider changing this to #!/bin/sh for better compatibility with Alpine's default shell, or document that bash needs to be installed first. The other distribution scripts (Ubuntu, Debian, RHEL) correctly use #!/bin/bash because they use bash-specific features like source and bash parameter expansion.

Suggested change
#!/bin/bash
#!/bin/sh

Copilot uses AI. Check for mistakes.
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 0f22f0d:

✅ Validation status: passed

File Status Preview URL Details
reference/docs-conceptual/install/install-alpine.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/install/install-debian.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/install/install-rhel.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/install/install-ubuntu.md ✅Succeeded View (>=powershell-5.1)

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 23027c2:

✅ Validation status: passed

File Status Preview URL Details
reference/5.1/Microsoft.PowerShell.Management/Set-Service.md ✅Succeeded View (powershell-5.1)
reference/7.4/Microsoft.PowerShell.Management/Set-Service.md ✅Succeeded View (powershell-7.4)
reference/7.5/Microsoft.PowerShell.Management/Set-Service.md ✅Succeeded View (powershell-7.5)
reference/7.6/Microsoft.PowerShell.Management/Set-Service.md ✅Succeeded View (powershell-7.6)
reference/docs-conceptual/install/install-alpine.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/install/install-debian.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/install/install-rhel.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/install/install-ubuntu.md ✅Succeeded View (>=powershell-5.1)

For more details, please refer to the build report.

@michaeltlombardi michaeltlombardi merged commit c97c99a into MicrosoftDocs:main Feb 4, 2026
7 checks passed
@sdwheeler sdwheeler deleted the sdw-w550714-shebang branch February 10, 2026 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants