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 action #75

Merged
merged 3 commits into from
Oct 15, 2023
Merged

Add winget action #75

merged 3 commits into from
Oct 15, 2023

Conversation

jimaek
Copy link
Member

@jimaek jimaek commented Jul 10, 2023

We're already present here https://github.com/microsoft/winget-pkgs/tree/master/manifests/j/jsdelivr/Globalping so I want to add auto updates.

Repo forked and token added.

@ayuhito since you were looking into this can you please check this PR?

@jimaek jimaek requested a review from ayuhito July 10, 2023 14:54
@jimaek jimaek linked an issue Jul 10, 2023 that may be closed by this pull request
Copy link
Member

@ayuhito ayuhito left a comment

Choose a reason for hiding this comment

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

It looks good to me in general but I was wondering why you didn't choose the GoReleaser variant of the packager?

The end result would be the same, but I think the GoReleaser variants add much more metadata to the manifest and would be a more cohesive experience.

.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
@jimaek
Copy link
Member Author

jimaek commented Jul 11, 2023

It looks good to me in general but I was wondering why you didn't choose the GoReleaser variant of the packager?

Its less automated, this action syncs the fork automatically and handles everything

@ayuhito
Copy link
Member

ayuhito commented Oct 15, 2023

@jimaek, I think this should now work theoretically and it works locally for me. Just need to run it with all the proper tokens.

@jimaek jimaek merged commit 5d140a8 into master Oct 15, 2023
1 check passed
@jimaek jimaek deleted the winget branch October 15, 2023 16:13
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.

Release new updates to Winget
2 participants