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

Winget-release github action #169

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jo-chemla
Copy link

Description:

Checklist:

  • Tests included.
  • Lint and formatter run with no errors.
  • Tests all pass.

@jo-chemla
Copy link
Author

According to the list of recent Github Actions runs, it looks that a maintainer needs to validate the winget-release. Not as mentioned in the original linked issue #168 that when merging this PR, you will need to proceed with a few steps for the final workflow to run smoothly and automatically on every github release:

  • You will need to create a classic Personal Access Token (PAT) with public_repo scope.
  • This token should be added as a repository secret by creating a new repo secret with name WINGET_ACC_TOKEN and value the token
  • Fork microsoft/winget-pkgs under the same account/organization as the project's repository. If you are forking winget-pkgs on a different account (e.g. bot/personal account), you can use the fork-user input to specify the username of the account where the fork is present.

image

@Jacalz
Copy link
Owner

Jacalz commented Nov 12, 2024

Sorry for the slow response. I have been rather busy and forgot about reviewing this. Will have a look as soon as I have some time over. Thanks for working on this.

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