Skip to content

New version: seerge.g-helper version 0.239#336279

Closed
Stensel8 wants to merge 1 commit intomicrosoft:masterfrom
Stensel8:g-helper-0239
Closed

New version: seerge.g-helper version 0.239#336279
Stensel8 wants to merge 1 commit intomicrosoft:masterfrom
Stensel8:g-helper-0239

Conversation

@Stensel8
Copy link
Contributor

@Stensel8 Stensel8 commented Feb 3, 2026

Checklist for Pull Requests

  • Have you signed the Contributor License Agreement?
  • Is there a linked Issue? If so, fill in the Issue number below.
    • Resolves #N/A (new version release)

Manifests

  • Have you checked that there aren't other open pull requests for the same manifest update/change?
  • This PR only modifies one (1) manifest
  • Have you validated your manifest locally with winget validate --manifest <path>?
  • Have you tested your manifest locally with winget install --manifest <path>? - No, because that will probably break my existing Zephyrus G16 configs.
  • Does your manifest conform to the 1.10 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Copilot AI review requested due to automatic review settings February 3, 2026 21:13
@wingetbot
Copy link
Collaborator

Validation Pipeline Run WinGetSvc-Validation-130-336279-20260203-1

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds WinGet manifests for seerge.g-helper v0.239.

Changes:

  • Added the version manifest for 0.239.
  • Added the default locale metadata (en-US), including release notes and URLs.
  • Added the installer manifest targeting the v0.239 GitHub release and SHA256.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
manifests/s/seerge/g-helper/0.239/seerge.g-helper.yaml Introduces the v0.239 version manifest tying the set together.
manifests/s/seerge/g-helper/0.239/seerge.g-helper.locale.en-US.yaml Provides package metadata and release notes for v0.239.
manifests/s/seerge/g-helper/0.239/seerge.g-helper.installer.yaml Defines the portable installer URL/SHA256 and dependency for v0.239.

PackageName: g-helper
PackageUrl: https://github.com/seerge/g-helper
License: GPL-3.0
LicenseUrl: https://github.com/seerge/g-helper/blob/HEAD/LICENSE
Copy link

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

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

LicenseUrl points to blob/HEAD/LICENSE, which is not version-specific. The manifest schema guidance recommends linking the license file to the exact package version (e.g., a tag/commit for v0.239) because licenses can change over time.

Suggested change
LicenseUrl: https://github.com/seerge/g-helper/blob/HEAD/LICENSE
LicenseUrl: https://github.com/seerge/g-helper/blob/v0.239/LICENSE

Copilot uses AI. Check for mistakes.
Comment on lines +8 to +10
NestedInstallerFiles:
- RelativeFilePath: GHelper.exe
PortableCommandAlias: GHelper
Copy link

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

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

PortableCommandAlias is specified under NestedInstallerFiles, but NestedInstallerType is not set. Per the manifest schema docs, PortableCommandAlias is only valid when NestedInstallerType is portable (and nested installers are intended for archive installer types like .zip). As written, the alias may be ignored by clients/validation tooling. Consider either (1) removing NestedInstallerFiles and relying on the portable executable name/Commands, or (2) if the download is actually an archive, set InstallerType to the archive type and add NestedInstallerType: portable.

Suggested change
NestedInstallerFiles:
- RelativeFilePath: GHelper.exe
PortableCommandAlias: GHelper

Copilot uses AI. Check for mistakes.
@wingetbot
Copy link
Collaborator

Found duplicate pull request(s)

@Stensel8 Stensel8 closed this Feb 3, 2026
@Stensel8 Stensel8 deleted the g-helper-0239 branch February 3, 2026 21:36
@Stensel8 Stensel8 restored the g-helper-0239 branch February 3, 2026 21:36
@Stensel8
Copy link
Contributor Author

Stensel8 commented Feb 3, 2026

Ah, my bad. Removed the duplicate.

@Stensel8 Stensel8 deleted the g-helper-0239 branch February 3, 2026 21:37
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.

2 participants