Skip to content

Fix netstd CI .NET SDK setup#3501

Merged
Jens-G merged 1 commit into
apache:masterfrom
kpumuk:setup-dotnet
May 19, 2026
Merged

Fix netstd CI .NET SDK setup#3501
Jens-G merged 1 commit into
apache:masterfrom
kpumuk:setup-dotnet

Conversation

@kpumuk
Copy link
Copy Markdown
Member

@kpumuk kpumuk commented May 19, 2026

Use the pinned setup-dotnet action instead of downloading and verifying the dotnet-install script manually. The manual path fails when the hosted script signature does not match the downloaded script. https://github.com/apache/thrift/actions/runs/26131051709/job/76858059843?pr=3500

  • Did you create an Apache Jira ticket? (Request account here, not required for trivial changes)
  • If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
  • Did you squash your changes to a single commit? (not required, but preferred)
  • Did you do your best to avoid breaking changes? If one was needed, did you label the Jira ticket with "Breaking-Change"?
  • If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

Use the pinned setup-dotnet action instead of downloading and verifying the dotnet-install script manually. The manual path fails when the hosted script signature does not match the downloaded script.

Co-Authored-By: OpenAI Codex (GPT-5.4) <codex@openai.com>
@mergeable mergeable Bot added the github_actions Pull requests that update GitHub Actions code label May 19, 2026
@kpumuk kpumuk marked this pull request as ready for review May 19, 2026 23:43
@kpumuk kpumuk requested review from Jens-G, fishy and jimexist as code owners May 19, 2026 23:43
@Jens-G Jens-G merged commit fbdd2d9 into apache:master May 19, 2026
98 of 99 checks passed
@kpumuk kpumuk deleted the setup-dotnet branch May 19, 2026 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants