-
Notifications
You must be signed in to change notification settings - Fork 69
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
Release Windows packages to winget-pkgs #1144
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1144 +/- ##
==========================================
- Coverage 49.46% 49.44% -0.02%
==========================================
Files 281 281
Lines 10726 10726
==========================================
- Hits 5306 5304 -2
- Misses 4855 4857 +2
Partials 565 565 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
.github/workflows/release.yml
Outdated
runs-on: windows-latest | ||
environment: release | ||
steps: | ||
- uses: vedantmgoyal2009/winget-releaser@v2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's pin the commit sha instead of using a tag.
Let's wait to merge this before hearing back from legal. I think it is OK since 1) we're not incorporating code from that project, 2) we're not statically or dynamically linking to any artifacts from that project, and 3) we're not modifying any code from that project. |
This has been approved by legal, so we are good to merge. |
Addressed feedback and discussed offline.
This reverts commit 2f1b81c.
This reverts commit 3dbb205.
Changes
This PR adds a release workflow which will automatically publish the CLI to winget-pkgs whenever a release is made. It uses https://github.com/vedantmgoyal2009/winget-releaser to release the windows binaries. @Exorcism0666 has been graciously making releases on our behalf, but we can do this automatically ourselves after this PR.
Tests