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

Add Winget Releaser workflow #276

Merged
merged 1 commit into from
Apr 9, 2023
Merged

Add Winget Releaser workflow #276

merged 1 commit into from
Apr 9, 2023

Conversation

sitiom
Copy link
Contributor

@sitiom sitiom commented Feb 13, 2023

This action automatically generates manifests for Winget Community Repository (microsoft/winget-pkgs) and submits them.

Before merging this:

  1. Add a classic PAT with public_repo scope as a repository secret named WINGET_TOKEN.

example

  1. Fork https://github.com/microsoft/winget-pkgs under @rhysd. The action will use that fork for making a branch and creating a PR with the upstream winget-pkgs repository on every release.
  2. Install Pull on the winget-pkgs fork to ensure that it is constantly updated.

You can test out the action inputs in this playground.

If you want to see an example of a PR created using this action, see microsoft/winget-pkgs/pulls (Pull request has been created with WinGet Releaser).

@rhysd
Copy link
Owner

rhysd commented Feb 23, 2023

@sitiom Can you remove 4ec176a? It is not related to this PR. I think we should focus on winget release workflow only in this PR.

@sitiom
Copy link
Contributor Author

sitiom commented Apr 4, 2023

@rhysd Updates on this? I have manually created a PR on the new version for now: microsoft/winget-pkgs#101511

@rhysd
Copy link
Owner

rhysd commented Apr 5, 2023

Ah, I'm sorry for this issue remaining open. Due to my personal reason, it is difficult to respond any issues/PRs for now. But I'll take a look at this within this week.

@rhysd rhysd merged commit 2eb188b into rhysd:main Apr 9, 2023
@rhysd
Copy link
Owner

rhysd commented Apr 9, 2023

I setup a token and fork repository. I'll try the first releaese.

@sitiom sitiom deleted the winget-releaser branch April 9, 2023 13:22
@sitiom
Copy link
Contributor Author

sitiom commented Apr 9, 2023

Did you install the Pull app on your fork? https://pull.git.ci/process/rhysd/winget-pkgs is showing "Not found."

@rhysd
Copy link
Owner

rhysd commented Apr 9, 2023

@sitiom I ran the action and the PR was created. If you have time, would you check the manifest submitted in the PR is correct? I'm not familiar with the manifest scheme.

microsoft/winget-pkgs#102052

@sitiom
Copy link
Contributor Author

sitiom commented Apr 9, 2023

@sitiom I ran the action and the PR was created. If you have time, would you check the manifest submitted in the PR is correct? I'm not familiar with the manifest scheme.

microsoft/winget-pkgs#102052

Looks good. The CI should fail if there is some error installing the manifest.

@rhysd
Copy link
Owner

rhysd commented Apr 9, 2023

OK, thanks for checking that. It seems to work fine.

@sitiom
Copy link
Contributor Author

sitiom commented Apr 15, 2023

@rhysd Wait, I just noticed that the version of the manifest you created includes the v prefix, which is incorrect. I have created a PR to fix the separate Winget workflow: #293.

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.

2 participants