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

Updating release-winget according to manifest schema changes #332

Merged
merged 14 commits into from
Apr 29, 2021
33 changes: 18 additions & 15 deletions .github/workflows/release-winget.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,26 +7,29 @@ jobs:
release:
runs-on: ubuntu-latest
steps:
- name: Update winget repository
uses: mjcheetham/update-winget@v1.0
- id: update-winget
name: Update winget repository
uses: mjcheetham/update-winget@v1.2
with:
token: ${{ secrets.WINGET_TOKEN }}
repo: microsoft/winget-pkgs
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we still want repo: microsoft/winget-pkgs, right? Or, is that built-in to v1.2?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's the default! However, we can keep if it's better to be explicit.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks! Using the default works for me.

This comment was marked as spam.

id: Microsoft.GitCredentialManagerCore
token: ${{ secrets.WINGET_TOKEN }}
releaseAsset: gcmcore-win-x86-(.*)\.exe
manifestText: |
Id: {{id}}
Version: {{version}}
Name: Git Credential Manager Core
PackageIdentifier: {{id}}
PackageVersion: {{version}}
PackageName: Git Credential Manager Core
Publisher: Microsoft Corporation
AppMoniker: git-credential-manager-core
Homepage: https://aka.ms/gcmcore
Tags: "gcm, gcmcore, git, credential"
Moniker: git-credential-manager-core
PackageUrl: https://aka.ms/gcmcore
Tags: [ gcm, gcmcore, git, credential ]
License: Copyright (C) Microsoft Corporation
Description: Secure, cross-platform Git credential storage with authentication to GitHub, Azure Repos, and other popular Git hosting services.
ShortDescription: Secure, cross-platform Git credential storage with authentication to GitHub, Azure Repos, and other popular Git hosting services.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there now a longer Description: that we should populate somehow?

Copy link
Contributor Author

@ldennington ldennington Apr 29, 2021

Choose a reason for hiding this comment

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

Description does still exist. However, for reasons that aren't super clear, ShortDescription is now required, while Description is not. I'm happy to take a stab at adding a longer Description though!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is there anything specific we'd want to call out in a longer Description?

Copy link
Contributor

Choose a reason for hiding this comment

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

I think we can wait on Description until we know more about where it will show up.

Installers:
- Arch: x86
Url: {{url}}
InstallerType: Inno
Sha256: {{sha256}}
- Architecture: x86
InstallerUrl: {{url}}
InstallerType: inno
InstallerSha256: {{sha256}}
PackageLocale: en-US
ManifestType: singleton
ManifestVersion: 1.0.0
alwaysUsePullRequest: true