Skip to content

Remove comments from workflow to avoid spell check flag#16215

Merged
chidozieononiwu merged 1 commit into
mainfrom
update-verify-links
Jun 27, 2026
Merged

Remove comments from workflow to avoid spell check flag#16215
chidozieononiwu merged 1 commit into
mainfrom
update-verify-links

Conversation

@chidozieononiwu

Copy link
Copy Markdown
Member

This pull request simplifies the .github/workflows/verify-links.yml workflow file by removing outdated and redundant comments. The core logic and functionality of the workflow remain unchanged.

Workflow cleanup:

  • Removed legacy comments describing the workflow's triggers and context, which were no longer necessary.
  • Removed an outdated comment about running the job after the Azure DevOps Compliance job.
  • Deleted comments explaining why full git history is fetched and why certain environment variables are set for the link verification script, as these are now considered self-explanatory or unnecessary.

Copilot AI review requested due to automatic review settings June 27, 2026 06:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This pull request cleans up the verify-links.yml GitHub Actions workflow by removing legacy/outdated inline comments, aiming to avoid spell-check noise while keeping the workflow behavior unchanged.

Changes:

  • Removed the header block comment describing historical context and trigger semantics.
  • Removed inline comments on the job if: gate and checkout/link-check steps.
  • No functional YAML keys/values were modified.

chidozieononiwu added a commit to Azure/azure-sdk-for-js that referenced this pull request Jun 27, 2026
…39107)

Sync .github/workflows directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#16215 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
chidozieononiwu added a commit to Azure/azure-sdk-for-rust that referenced this pull request Jun 27, 2026
…4672)

Sync .github/workflows directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#16215 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
@chidozieononiwu chidozieononiwu merged commit 1ff32b0 into main Jun 27, 2026
8 of 10 checks passed
@chidozieononiwu chidozieononiwu deleted the update-verify-links branch June 27, 2026 06:42
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