Skip to content

Add winget release script - #15050

Merged
Aditya Patwardhan (adityapatwardhan) merged 9 commits into
PowerShell:masterfrom
rjmholt:winget-release
Apr 16, 2021
Merged

Add winget release script#15050
Aditya Patwardhan (adityapatwardhan) merged 9 commits into
PowerShell:masterfrom
rjmholt:winget-release

Conversation

@rjmholt

Copy link
Copy Markdown
Collaborator

PR Summary

Adds a script to automatically generate winget release PRs, such as microsoft/winget-pkgs#8854.

PR Context

Eventually we should be able to use this PR in automation, using a PowerShell GitHub account.

PR Checklist

Comment thread tools/releaseToWinget.ps1 Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe make this mandatory

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

If you run it yourself with PowerShellForGitHub and don't provide a token, it will use its locally stored token

@ghost ghost added the Review - Needed The PR is being reviewed label Mar 24, 2021
@ghost

Copy link
Copy Markdown

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

@adityapatwardhan

Copy link
Copy Markdown
Member

Rob Holt (@rjmholt) do you also want to make changes to the release pipline in this PR too?

@ghost ghost removed the Review - Needed The PR is being reviewed label Apr 1, 2021
@rjmholt

Copy link
Copy Markdown
Collaborator Author

Rob Holt (@rjmholt) do you also want to make changes to the release pipline in this PR too?

I'd prefer to do that in a separate PR, since that will require more work and time to test properly

@rjmholt

Copy link
Copy Markdown
Collaborator Author

Aditya Patwardhan (@adityapatwardhan) I believe this is ready to merge

@rjmholt

Copy link
Copy Markdown
Collaborator Author

Aditya Patwardhan (@adityapatwardhan) scratch that, it looks like there's something to fix still

@rjmholt
Rob Holt (rjmholt) marked this pull request as draft April 15, 2021 02:09
@rjmholt
Rob Holt (rjmholt) marked this pull request as ready for review April 15, 2021 03:01
@rjmholt

Rob Holt (rjmholt) commented Apr 15, 2021

Copy link
Copy Markdown
Collaborator Author

Ok fixed. This is ready again now. See microsoft/winget-pkgs#10754 for an example PR

@rjmholt

Copy link
Copy Markdown
Collaborator Author

Those markdown link checks are failing again. We should really kick those tests out to only run them on nightlies or something.

@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) merged commit 80f685f into PowerShell:master Apr 16, 2021
@adityapatwardhan Aditya Patwardhan (adityapatwardhan) added the CL-Tools Indicates that a PR should be marked as a tools change in the Change Log label Apr 16, 2021
@rjmholt
Rob Holt (rjmholt) deleted the winget-release branch April 16, 2021 22:19
Keith Hill (rkeithhill) pushed a commit to rkeithhill/PowerShell that referenced this pull request May 3, 2021
@ghost

Copy link
Copy Markdown

🎉v7.2.0-preview.6 has been released which incorporates this pull request.:tada:

Handy links:

Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-Tools Indicates that a PR should be marked as a tools change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants