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.

Sign up for GitHub

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

Jump to bottom

New version: GIGABYTE.GigabyteControlCenter version 25.02.07.01 #230284

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

New version: GIGABYTE.GigabyteControlCenter version 25.02.07.01 #230284

wants to merge 4 commits into from

Conversation

Dragon1573
Copy link
Contributor

@Dragon1573 Dragon1573 commented Feb 16, 2025 •
edited by microsoft-github-policy-service bot
Loading

Checklist for Pull Requests

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>?
  • Does your manifest conform to the 1.9 schema?

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

Manual validation

Image

Microsoft Reviewers: Open in CodeFlow

@wingetbot
Copy link
Collaborator

Service Badge  Service Badge  

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@wingetbot
Copy link
Collaborator

Url Validation Error

@wingetbot wingetbot added the Validation-Installation-Error Error installing application label Feb 16, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Feb 16, 2025
@DuckDuckStudio
Copy link
Contributor

DuckDuckStudio commented Feb 17, 2025 •
edited
Loading

Url Validation Error

It looks like it's a false alarm or a regional block.
image
image
When you visit https://www.gigabyte.com in mainland China, you will be automatically redirected to https://www.gigabyte.cn.

@Dragon1573 @DuckDuckStudio
Apply suggestions from code review
f000099 
- Remove Mainland China specific installer entrance
  - The unified ".com" domain is not blocked in China
  - In Mainland China, ".com" will be automatically redirected to ".cn"
  - ".cn" connection speed is slightly slower than ".com"

Co-authored-by: 鸭鸭「カモ」 <[email protected]>
@wingetbot
Copy link
Collaborator

/AzurePipelines run

@microsoft-github-policy-service microsoft-github-policy-service bot removed Needs-Author-Feedback This needs a response from the author. Validation-Installation-Error Error installing application labels Feb 17, 2025
@Dragon1573 @DuckDuckStudio
Apply suggestions from code review
12939fa 
- Correction of "DisplayVersion" property to match that in screenshot

Co-authored-by: 鸭鸭「カモ」 <[email protected]>
@wingetbot
Copy link
Collaborator

/AzurePipelines run

@wingetbot
Copy link
Collaborator

Url Validation Error

@wingetbot wingetbot added the Validation-Installation-Error Error installing application label Feb 17, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Feb 17, 2025
@Dragon1573
Copy link
Contributor Author

Hello @Dragon1573,

The package manager bot determined there was an issue with installing the application correctly. Please check the application installs correctly. Once repaired, please push an update to your pull request.

Template: msftbot/validationError/installation/general

This problem should be suppressed.

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels Feb 17, 2025
@stephengillie
Copy link
Collaborator

Automatic Validation ended with:

Installation failed with exit code -1978335229 2025-02-17 01:08:06.394 [CLI ] Caught std::exception: bad allocation

Automated error analysis suggests -1978335229 may mean Executing command failed

(Automated response - build 980.)

@stephengillie
Copy link
Collaborator

Hello @Dragon1573,
The package manager bot determined there was an issue with installing the application correctly. Please check the application installs correctly. Once repaired, please push an update to your pull request.
Template: msftbot/validationError/installation/general

This problem should be suppressed.

@microsoft-github-policy-service microsoft-github-policy-service bot added Changes-Requested Changes Requested Needs-Author-Feedback This needs a response from the author. labels Feb 17, 2025
@Dragon1573 @DuckDuckStudio
Apply suggestions from code review
a7ecb58 
- Switch domain suffix of the package
- Prevent false positive report of region blocking

Co-authored-by: 鸭鸭「カモ」 <[email protected]>
@wingetbot
Copy link
Collaborator

/AzurePipelines run

@microsoft-github-policy-service microsoft-github-policy-service bot removed Needs-Attention This work item needs to be reviewed by a member of the core team. Needs-Author-Feedback This needs a response from the author. Validation-Installation-Error Error installing application Changes-Requested Changes Requested labels Feb 18, 2025
@wingetbot wingetbot added the Validation-Installation-Error Error installing application label Feb 18, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Feb 18, 2025
@Dragon1573 Dragon1573 removed their assignment Feb 18, 2025
@DuckDuckStudio
Copy link
Contributor

Automatic Validation ended with:

Installation failed with exit code -1978335229 2025-02-17 01:08:06.394 [CLI ] Caught std::exception: bad allocation

Automated error analysis suggests -1978335229 may mean Executing command failed

(Automated response - build 980.)

I don't think this package executes by command...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DuckDuckStudio DuckDuckStudio DuckDuckStudio left review comments

@stephengillie stephengillie stephengillie requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
Needs-Author-Feedback This needs a response from the author. Validation-Installation-Error Error installing application
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Update Request]: GIGABYTE.GigabyteControlCenter version 25.02.07.01
4 participants
@Dragon1573 @wingetbot @DuckDuckStudio @stephengillie