Skip to content

Sync eng/common directory with azure-sdk-tools for PR 13281#54413

Merged
weshaggard merged 3 commits intomainfrom
sync-eng/common-FixNpmLinkChecks-13281
Dec 10, 2025
Merged

Sync eng/common directory with azure-sdk-tools for PR 13281#54413
weshaggard merged 3 commits intomainfrom
sync-eng/common-FixNpmLinkChecks-13281

Conversation

@azure-sdk
Copy link
Copy Markdown
Collaborator

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#13281 See eng/common workflow

Copilot AI review requested due to automatic review settings December 9, 2025 22:34
@azure-sdk azure-sdk requested a review from a team as a code owner December 9, 2025 22:34
@azure-sdk azure-sdk requested a review from weshaggard December 9, 2025 22:34
@azure-sdk azure-sdk added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Dec 9, 2025
Copy link
Copy Markdown
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 syncs the eng/common directory with azure-sdk-tools PR 13281, adding support for verifying npmjs.com package links.

  • Adds special handling for npmjs.com links to work around Cloudflare 403 errors by using the npm registry API instead

weshaggard and others added 3 commits December 10, 2025 19:45
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@azure-sdk azure-sdk force-pushed the sync-eng/common-FixNpmLinkChecks-13281 branch from dfcba5c to 35d360c Compare December 10, 2025 19:45
@weshaggard weshaggard merged commit 2719014 into main Dec 10, 2025
18 of 21 checks passed
@weshaggard weshaggard deleted the sync-eng/common-FixNpmLinkChecks-13281 branch December 10, 2025 20:43
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. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants