Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Dotnet from release step of GH workflow #11684

Merged
merged 2 commits into from Dec 20, 2022

Conversation

RobbieMcKinstry
Copy link
Contributor

Description

Now that dotnet has been exfiltrated to https://github.com/pulumi/pulumi-dotnet we do not release it as part of the release workflow. The release step errors since the dotnet artifacts are no longer uploaded to GitHub as part of the release bundle.

Is there an easy way to test this change?

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • I have run make changelog and committed the changelog/pending/<file> documenting my change
  • Yes, there are changes in this PR that warrants bumping the Pulumi Service API version

@pulumi-bot
Copy link
Contributor

pulumi-bot commented Dec 19, 2022

Changelog

[uncommitted] (2022-12-20)

@RobbieMcKinstry RobbieMcKinstry added the impact/no-changelog-required This issue doesn't require a CHANGELOG update label Dec 19, 2022
Copy link
Member

@AaronFriel AaronFriel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small change requested, will LGTM after that.

@@ -42,7 +42,6 @@ on:
# Example provided for illustration, this value is derived by scripts/get-job-matrix.py build
default: |
{
"dotnet": "6.0.x",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be retained, it's an example value and it's kept consistent across all of our workflows. If this is removed, someone might incorrectly believe the dotnet key isn't available here.

@AaronFriel
Copy link
Member

AaronFriel commented Dec 19, 2022

@RobbieMcKinstry pushing snapshot versions to package managers again is blocked by #10627.

Summary: I tried to run our entire workflow for a prerelease version and this was unpopular - it caused us to push a prerelease to our Docs pipeline, Homebrew, and others.

@AaronFriel
Copy link
Member

bors merge

@bors
Copy link
Contributor

bors bot commented Dec 20, 2022

Build succeeded:

@bors bors bot merged commit 0f23e35 into master Dec 20, 2022
@bors bors bot deleted the mckinstry/remove-release-dotnet branch December 20, 2022 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/no-changelog-required This issue doesn't require a CHANGELOG update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants