Skip to content

Conversation

@mdanish-kh
Copy link
Contributor

@mdanish-kh mdanish-kh commented Jan 4, 2025

This PR proposes to add a GitHub action for submitting the latest stable / beta release to WinGet as it gets published. microsoft/winget-create is used as the tool for submitting the latest package.

Steps needed from maintainers

If the maintainers approve of these changes, they will need to do the following before merging this PR:

  1. Fork microsoft/winget-pkgs under a personal or bot account.
  2. Create a public access token (classic) with public_repo scope from the user account where the fork exists.
  3. Store the created token in a repository secret here with the name WINGET_GITHUB_TOKEN

Once the fork is created, I'll update the fork name in the "Sync winget-pkgs fork" step

For reference, maintainers may see similar implemented actions in the following repos:
Terminal, DevHome, PowerToys, Oh-my-posh

@mdanish-kh mdanish-kh requested a review from a team January 4, 2025 19:38
@mdanish-kh mdanish-kh force-pushed the winget-ci branch 2 times, most recently from 6dfb80c to 49ea940 Compare January 4, 2025 19:53
@waldekmastykarz
Copy link
Collaborator

Thank you for the suggestion! Let us check it out and we'll get back to you asap

@garrytrinder
Copy link
Contributor

Thanks for the PR @mdanish-kh

This all looks good to me and greatly simplifies the way in which we publish new versions of Dev Proxy to winget, thank you!

I'm happy to use the fork under my account for this which can be found at https://github.com/garrytrinder/winget-pkgs/, I've also created the WINGET_GITHUB_TOKEN secret in this repo.

@mdanish-kh
Copy link
Contributor Author

Thank you @garrytrinder. I updated the fork name in the action, so the PR should be ready to go from my end 🙂

@garrytrinder garrytrinder merged commit 90990fa into dotnet:main Jan 8, 2025
4 checks passed
@garrytrinder
Copy link
Contributor

Thank you @mdanish-kh ❤️

@mdanish-kh mdanish-kh deleted the winget-ci branch January 8, 2025 10:51
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