Skip to content

Conversation

@azure-sdk
Copy link
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
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 changes from azure-sdk-tools PR #13281 to add special handling for npmjs.com package links in the link verification script. The change works around Cloudflare-induced 403 errors by redirecting npmjs.com package verification to use the npm registry API instead.

Key Changes

  • Added regex pattern matching for npmjs.com package URLs in the ProcessLink function
  • Implemented ProcessNpmLink function that converts npmjs.com package URLs to registry.npmjs.org API URLs for verification
  • Follows the existing pattern established by ProcessCratesIoLink for handling domains with special verification requirements

@azure-sdk azure-sdk force-pushed the sync-eng/common-FixNpmLinkChecks-13281 branch from 070e3d8 to 7d7914e Compare December 10, 2025 19:46
@azure-sdk azure-sdk merged commit a0428ba into main Dec 10, 2025
16 checks passed
@azure-sdk azure-sdk deleted the sync-eng/common-FixNpmLinkChecks-13281 branch December 10, 2025 20:47
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