Skip to content

Conversation

@danieljurek
Copy link
Member

@danieljurek danieljurek commented Nov 5, 2025

C++ docs onboarding doesn't have a legacy moniker and needs to skip.

Partial: Azure/azure-sdk-for-cpp#6642

@danieljurek danieljurek requested a review from a team as a code owner November 5, 2025 22:11
Copilot AI review requested due to automatic review settings November 5, 2025 22:11
@danieljurek danieljurek self-assigned this Nov 5, 2025
@danieljurek danieljurek moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 📆🎇 Nov 5, 2025
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 the ability to skip legacy package onboarding in the documentation update process. A new optional switch parameter $SkipLegacyOnboarding is introduced that allows callers to exclude the 'legacy' moniker from the documentation onboarding workflow.

Key Changes:

  • Added a new switch parameter $SkipLegacyOnboarding to the script
  • Made the 'legacy' moniker conditional based on the new parameter

@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

@danieljurek danieljurek enabled auto-merge (squash) November 6, 2025 23:25
azure-sdk added a commit to Azure/azure-sdk-for-js that referenced this pull request Nov 6, 2025
azure-sdk added a commit to Azure/azure-sdk-for-rust that referenced this pull request Nov 6, 2025
@danieljurek danieljurek merged commit 2a05634 into main Nov 6, 2025
18 checks passed
@danieljurek danieljurek deleted the djurek/docs-skip-legacy-onboarding branch November 6, 2025 23:40
@kurtzeborn kurtzeborn moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 📆🎇 Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants