Skip to content

Conversation

@vedantmgoyal9
Copy link
Contributor

This action automatically generates manifests for WinGet Community Repository (microsoft/winget-pkgs) and submits them.

Before merging this:

  1. Please add a GitHub token with public_repo and workflow scopes as a repository secret and rename the secret name in the workflow.
  2. Fork microsoft/winget-pkgs under @sqlitebrowser.

@vedantmgoyal9
Copy link
Contributor Author

@chrisjlocke sorry for responding so late.

@justinclift
Copy link
Member

Interesting idea. @chrisjlocke You do Windows stuff... does this look good to you? 😄

@vedantmgoyal9
Copy link
Contributor Author

/cc @chrisjlocke

@chrisjlocke
Copy link
Member

add a token with public_repo and workflow stats

image

Did this, but clicking 'workflow' automatically ticked 'public_repo' (well, the whole section) .. assume that's OK?

@chrisjlocke
Copy link
Member

does this look good to you?

WinGet is great for installing stuff - the Windows version of appt ... so an easy way to get people to install, rather than give them a link, tell them to extract, blah blah ... the trouble is, WinGet is relatively new and fighting with chocolatey. Netscape vs Explorer war! ;) Guess WinGet will win as it'll come installed by default on Windows, unlike Chocolatey which has to be installed...defeating the purpose of making an app easy to install by having to install the installer.... ;)

Created a token, but as its my user, is that good enough?

@vedantmgoyal9
Copy link
Contributor Author

assume that's OK?

Yes, it's fine. I guess we're ready to merge this PR.

@lucydodo
Copy link
Member

lucydodo commented Sep 27, 2022

@chrisjlocke It looks like we need to add the generated token to our repository Action Secrets before merge this PR :)
Settings -> Security -> Secrets -> Actions

More information can be found here. :)

https://docs.github.com/en/actions/security-guides/encrypted-secrets

@chrisjlocke
Copy link
Member

Thanks @lucydodo 👍

@vedantmgoyal9
Copy link
Contributor Author

Can this be merged now?

@chrisjlocke chrisjlocke merged commit 8cbf579 into sqlitebrowser:master Oct 7, 2022
@chrisjlocke
Copy link
Member

Merged. 😀

Thanks for your contribution. :)

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.

[Feature]: Publish to Windows Package Managar (WinGet)

4 participants