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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Winget Releaser workflow #537

Closed
wants to merge 1 commit into from
Closed

Add Winget Releaser workflow #537

wants to merge 1 commit into from

Conversation

sitiom
Copy link
Contributor

@sitiom sitiom commented Aug 1, 2022

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

I have already created a PR to add ElectronMail to Winget; this workflow will be used to update it.

Before merging this:

  1. Add a PAT with public_repo scope as a repository secret named WINGET_TOKEN (or rename the secret name in the workflow).

example

  1. Fork https://github.com/microsoft/winget-pkgs under @vladimiry. The action will use that fork for making a branch and creating a PR with the upstream winget-pkgs repository on every release.
  2. Sign the Contributor License Agreement for winget-pkgs as you (the PAT owner) will be a first-time contributor.

If you would like to read about this action further, the documentation is here, and the source code is here.

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

@vladimiry
Copy link
Owner

Thanks for the effort towards automating things and for the detailed instructions, but at this time I tend to reject the PR.

Currently, none of the listed in the README package repositories requires the project to include/maintain repo-specific stuff, and I prefer it to stay this way. So if some repo wants to include the app package, that would be an independent pulling strategy.

Besides, at the moment, there is no interest in signing some CLA.

CC @joshirio / @hiddeninthesand

@vladimiry vladimiry closed this Aug 1, 2022
@sitiom
Copy link
Contributor Author

sitiom commented Aug 2, 2022

@vladimiry You do maintain the flatpak package, right? What's the difference with this one?

Edit: I see. You maintain it manually, but it can use some automation since you control the package.

@vladimiry
Copy link
Owner

You do maintain the flatpak package, right? What's the difference with this one?

This is a different project which is pulling stuff from here. So technically, ElectronMail project knows nothing about it (pulling vs pushing strategy).

@sitiom sitiom deleted the winget-releaser branch August 2, 2022 08:52
@vladimiry vladimiry mentioned this pull request Aug 2, 2022
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.

None yet

2 participants