Skip to content

Add shebang and make cspell script executable for bash invocation#11330

Merged
benbp merged 4 commits intoAzure:mainfrom
benbp:benbp/spellcheck
Aug 11, 2025
Merged

Add shebang and make cspell script executable for bash invocation#11330
benbp merged 4 commits intoAzure:mainfrom
benbp:benbp/spellcheck

Conversation

@benbp
Copy link
Member

@benbp benbp commented Jul 24, 2025

Pulling in docs from @fanyang-mono's #11289 and updating the cspell script so that it can be invoked from a bash shell.

@benbp benbp added the Central-EngSys This issue is owned by the Engineering System team. label Jul 24, 2025
@benbp benbp self-assigned this Jul 24, 2025
Copilot AI review requested due to automatic review settings July 24, 2025 18:11
@benbp benbp added the Central-EngSys This issue is owned by the Engineering System team. label Jul 24, 2025
@benbp benbp requested a review from a team as a code owner July 24, 2025 18:12
@benbp benbp requested a review from weshaggard July 24, 2025 18:12
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 a shebang line and makes the cspell script executable to enable direct bash invocation. The changes allow the PowerShell script to be run directly from the command line without explicitly invoking PowerShell.

  • Adds shebang line to PowerShell script for direct execution
  • Includes comprehensive documentation for the spelling check scripts

Reviewed Changes

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

File Description
eng/common/spelling/README.md Adds complete documentation for the spelling check functionality
eng/common/spelling/Invoke-Cspell.ps1 Adds shebang line for direct script execution

@benbp benbp moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 📆🎇 Jul 24, 2025
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@azure-sdk
Copy link
Collaborator

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

@azure-sdk
Copy link
Collaborator

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

@benbp benbp enabled auto-merge (squash) August 11, 2025 18:50
azure-sdk added a commit to Azure/azure-sdk-for-js that referenced this pull request Aug 11, 2025
Sync eng/common directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#11330 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

---------

Co-authored-by: Fan Yang <yangfan@microsoft.com>
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
azure-sdk added a commit to Azure/azure-sdk-for-rust that referenced this pull request Aug 11, 2025
Sync eng/common directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#11330 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

---------

Co-authored-by: Fan Yang <yangfan@microsoft.com>
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@benbp benbp disabled auto-merge August 11, 2025 19:40
@benbp benbp merged commit 6135bcb into Azure:main Aug 11, 2025
7 checks passed
@kurtzeborn kurtzeborn moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 📆🎇 Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants