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 badge #538

Closed
wants to merge 2 commits into from
Closed

Add Winget badge #538

wants to merge 2 commits into from

Conversation

sitiom
Copy link
Contributor

@sitiom sitiom commented Aug 2, 2022

Add Winget badge in README. I used a custom query to https://repology.org/project/electronmail/versions since shields.io doesn't support Winget badges yet. (badges/shields#5548)

@vladimiry
Copy link
Owner

vladimiry commented Aug 2, 2022

I will need to mention a repo maintainer (preferably GitHub username). Is this @sitiom?

@sitiom
Copy link
Contributor Author

sitiom commented Aug 2, 2022

There isn't a sense of a maintainer in https://github.com/microsoft/winget-pkgs as it is a central repository where anyone can contribute, but most automatic updates are handled by @vedantmgoyal2009 in his automation repo (https://github.com/vedantmgoyal2009/vedantmgoyal2009) or from software vendors themselves using the Winget Releaser action or something similar.

I have added ElectronMail to @vedantmgoyal2009's automation in vedantmgoyal9/vedantmgoyal9#709. This is how you would want it to be, right?

@vladimiry
Copy link
Owner

This is how would want it to be, right?

Yes, it appears so.

@vedantmgoyal9
Copy link

Can you please use winget-releaser? Due to >1000 packages, GitHub's API rate limit is exceeded and packages are not able to update. That's why I have developed an action so the software developers are able to maintain themselves and non-GitHub packages are able to update.

@vladimiry
Copy link
Owner

Can you please use winget-releaser?

This is a relatively small burden (automated stuff), but still at this time I'm not ready to accept it. See details in #537.

@vladimiry
Copy link
Owner

Merged manually, see 2043c52.

@vladimiry vladimiry closed this Aug 2, 2022
@sitiom sitiom deleted the patch-1 branch August 2, 2022 09: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.

None yet

3 participants