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 Releaser #1387

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

Add WinGet Releaser #1387

wants to merge 4 commits into from

Conversation

russellbanks
Copy link

This pull request adds a workflow that automatically publishes espanso to WinGet every release. Please do not merge this pull request without reading the below and making the required changes :)

  • token: ${{ secrets.WINGET_TOKEN }}

This is the bit that will require action from the person whose account will be used for the pull requests (a member of the espanso GitHub organisation). This is a GitHub token with which the action will authenticate with GitHub and create a pull request on the winget-pkgs repository.

To do this, you will need to:

  1. Create a Personal Access Token (PAT) with public_repo scope.

image

  1. Create a repository secret containing the token. Super important: call this token: WINGET_TOKEN.
    See using encrypted secrets in a workflow for more details.

Finally, a fork of winget-pkgs needs to be created under the espanso organisation. Please do not delete the fork after a pull request has been merged as the action will use that fork for making a branch and merging it with the upstream winget-pkgs repository on every espanso release.

If you would like to read about this action further, the source code and documentation are here.

Requesting review from @vedantmgoyal2009

.github/workflows/release.yml Outdated Show resolved Hide resolved
@vedantmgoyal9
Copy link

/cc @federico-terzi

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

Hi! interesting! there are a couple of conflicts to resolve. Can you make it? 💘 Thank you!

Copy link

@vedantmgoyal9 vedantmgoyal9 left a comment

Choose a reason for hiding this comment

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

@russellbanks please resolve merge conflicts.

@federico-terzi
Copy link
Collaborator

Thanks! Before proceeding with this, I'd like to review how the Github Action for the release work :) I'll try to do it soon

Sorry for the late reply!

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

4 participants