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

Adding GitHub action for winget-releaser #1115

Merged
merged 0 commits into from
Oct 23, 2024
Merged

Adding GitHub action for winget-releaser #1115

merged 0 commits into from
Oct 23, 2024

Conversation

assarbad
Copy link
Collaborator

Summary:

This would conclude #848 (which also requires microsoft/winget-pkgs#184453 to be merged).

Description for the changelog:

  • It is now possible to install OWASP Threat Dragon with WinGet, using winget install -e --id OWASP.ThreatDragon or subsequently keep it up-to-date with winget upgrade -e --id OWASP.ThreatDragon

Other info:

Below follows the commit message in full:


Instructions for prerequisites

The WINGET_TOKEN needs to be created and registered in the repo which does the release. For good measure we need to clone winget-pkgs on the same user (or org) or set via fork-user it as per the README for winget-releaser.

Configure the Pull app to keep the winget-pkgs fork in sync with its upstream.

That winget-pkgs fork is going to be the source of the PR filed at microsoft/winget-pkgs and the PRs are being opened on behalf of the owner of the used token.

  1. Head to https://github.com/settings/tokens and create a new (classic) personal access token with only the public_repo scope activated for it
  2. In https://github.com/OWASP/threat-dragon/settings/secrets/actions create a secret named WINGET_TOKEN (as per vedantmgoyal9/winget-releaser)
  3. Merge this PR
  4. Create a (subsequent) release

@jgadsden
Copy link
Collaborator

hello @assarbad , I have created WINGET_TOKEN which contains the personal access token

Copy link
Collaborator

@jgadsden jgadsden left a comment

Choose a reason for hiding this comment

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

thanks @assarbad , all looks good for merging - do we wait for microsoft/winget-pkgs#184453 to have been merged first?

@jgadsden jgadsden merged commit 3130fcb into OWASP:main Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

winget support would be great
2 participants