Skip to content

C#: nugetApiKey check on publishing#6887

Merged
jkschneider merged 1 commit intomainfrom
greg-disable-nuget-publish
Mar 5, 2026
Merged

C#: nugetApiKey check on publishing#6887
jkschneider merged 1 commit intomainfrom
greg-disable-nuget-publish

Conversation

@greg-at-moderne
Copy link
Copy Markdown
Contributor

@greg-at-moderne greg-at-moderne commented Mar 5, 2026 •
edited
Loading

What's changed?

Adding a condition to the publish task in rewrite-csharp.

What's your motivation?

Fix broken publishSnapshots flow which errors out with:

Caused by: org.gradle.api.GradleException: nugetApiKey property is required for NuGet publishing
	at Build_gradle$csharpPublish$2$1.execute(build.gradle.kts:170)

@greg-at-moderne greg-at-moderne self-assigned this Mar 5, 2026
@github-project-automation github-project-automation Bot moved this to In Progress in OpenRewrite Mar 5, 2026
@greg-at-moderne greg-at-moderne marked this pull request as ready for review March 5, 2026 15:23
@jkschneider jkschneider merged commit 8b7a44f into main Mar 5, 2026
1 check passed
@jkschneider jkschneider deleted the greg-disable-nuget-publish branch March 5, 2026 16:08
@github-project-automation github-project-automation Bot moved this from In Progress to Done in OpenRewrite Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@greg-at-moderne greg-at-moderne

Labels

None yet

Projects

OpenRewrite
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@greg-at-moderne @jkschneider
⚡