Skip to content

Use local checkout for fetching package.json instead of accessing github directly#9447

Open
dibarbet wants to merge 3 commits into
dotnet:mainfrom
dibarbet:fix-release-tagger
Open

Use local checkout for fetching package.json instead of accessing github directly#9447
dibarbet wants to merge 3 commits into
dotnet:mainfrom
dibarbet:fix-release-tagger

Conversation

@dibarbet

Copy link
Copy Markdown
Member

Release pipeline is failing to access raw.githubusercontent.com due to network blocks when generating tags. This modifies the pipeline to use the local checkout instead (note that the pipeline checks out vscode-csharp at the commit that is being released)

Copilot AI review requested due to automatic review settings June 18, 2026 00:23
@dibarbet dibarbet requested a review from a team as a code owner June 18, 2026 00:23

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

Updates the release tagging task to avoid fetching package.json from raw.githubusercontent.com (which can be blocked in restricted networks) by reading package.json from the locally checked-out repository or via git show for the specified release commit.

Changes:

  • Replaces remote fetch of package.json with local resolution via git rev-parse HEAD + git show <commit>:package.json.
  • Removes the dependency on githubPAT for determining the Roslyn component commit (NuGet inspection now only needs local repo access).
  • Adds small git helper utilities (getPackageJsonFromReleaseCommit, getGitOutput) to support local reads.
Show a summary per file
File Description
tasks/tags/createTags.ts Switches tag-generation inputs from GitHub raw content fetching to local checkout/git-object reads for package.json.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 1

Comment thread tasks/tags/createTags.ts
Comment thread tasks/tags/createTags.ts
Copilot AI review requested due to automatic review settings June 19, 2026 18: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.

Copilot's findings

  • Files reviewed: 2/2 changed files
  • Comments generated: 3

Comment thread .vscode/settings.json
Comment thread tasks/tags/createTags.ts
Comment thread tasks/tags/createTags.ts
@dibarbet dibarbet force-pushed the fix-release-tagger branch from 1163299 to be401d2 Compare June 19, 2026 18:32
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.

3 participants