Skip to content

Conversation

@rovabunw
Copy link
Contributor

@rovabunw rovabunw commented Dec 29, 2024

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.


Microsoft Reviewers: Open in CodeFlow

@wingetbot
Copy link
Collaborator

Service Badge  Service Badge  

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@wingetbot wingetbot added the Internal-Error-Dynamic-Scan The test for Dynamic Scanning in the Installation Validation failed. label Dec 29, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Attention This work item needs to be reviewed by a member of the core team. label Dec 29, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Retry-1 flag to indicate retried label Dec 29, 2024
@wingetbot
Copy link
Collaborator

/AzurePipelines run

@microsoft-github-policy-service microsoft-github-policy-service bot removed Internal-Error-Dynamic-Scan The test for Dynamic Scanning in the Installation Validation failed. Needs-Attention This work item needs to be reviewed by a member of the core team. labels Dec 29, 2024
@ItzLevvie
Copy link
Contributor

@wingetbot run

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@stephengillie
Copy link
Collaborator

Verify:
image
The application seems to be tray-based, but appeared to have exited. So I wasn't able to have it display version data.

@microsoft-github-policy-service microsoft-github-policy-service bot merged commit 51aa80b into microsoft:master Dec 30, 2024
1 of 2 checks passed
@microsoft-github-policy-service microsoft-github-policy-service bot added the Moderator-Approved One of the Moderators has reviewed and approved this PR label Dec 30, 2024
Comment on lines +13 to +18
AppsAndFeaturesEntries:
- DisplayName: Liquit Universal Agent for Windows (x64)
Publisher: Liquit Software B.V.
DisplayVersion: 4.2.3735.5913
ProductCode: '{F0D42568-BD85-4A9F-A330-1C095773BEE7}'
UpgradeCode: '{2B15E482-04E7-48A7-897A-12E5F877A2FF}'
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
AppsAndFeaturesEntries:
- DisplayName: Liquit Universal Agent for Windows (x64)
Publisher: Liquit Software B.V.
DisplayVersion: 4.2.3735.5913
ProductCode: '{F0D42568-BD85-4A9F-A330-1C095773BEE7}'
UpgradeCode: '{2B15E482-04E7-48A7-897A-12E5F877A2FF}'
AppsAndFeaturesEntries:
- DisplayName: Liquit Universal Agent for Windows (x64)
UpgradeCode: '{2B15E482-04E7-48A7-897A-12E5F877A2FF}'

Comment on lines +23 to +28
AppsAndFeaturesEntries:
- DisplayName: Liquit Universal Agent for Windows (x86)
Publisher: Liquit Software B.V.
DisplayVersion: 4.2.3735.5913
ProductCode: '{7BB1F40D-A8A0-4A66-A5D7-3AC0EE8A7651}'
UpgradeCode: '{2B15E482-04E7-48A7-897A-12E5F877A2FF}'
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
AppsAndFeaturesEntries:
- DisplayName: Liquit Universal Agent for Windows (x86)
Publisher: Liquit Software B.V.
DisplayVersion: 4.2.3735.5913
ProductCode: '{7BB1F40D-A8A0-4A66-A5D7-3AC0EE8A7651}'
UpgradeCode: '{2B15E482-04E7-48A7-897A-12E5F877A2FF}'
AppsAndFeaturesEntries:
- DisplayName: Liquit Universal Agent for Windows (x86)
UpgradeCode: '{2B15E482-04E7-48A7-897A-12E5F877A2FF}'

@mdanish-kh
Copy link
Contributor

@stephengillie - is there a way your automation can check for the condition DisplayVersion == PackageVersion and not approve it in that case?

@wingetbot
Copy link
Collaborator

Publish pipeline succeeded for this Pull Request. Once you refresh your index, this change should be present.

@stephengillie
Copy link
Collaborator

stephengillie commented Dec 30, 2024

Edit: Detection is working, just need to fix the canned message.

@ stephengillie - is there a way your automation can check for the condition DisplayVersion == PackageVersion and not approve it in that case?

Hi @mdanish-kh,

Yes, this is complete. I reused the old AnF check.

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

Labels

Changes-Requested Changes Requested Moderator-Approved One of the Moderators has reviewed and approved this PR Needs-Author-Feedback This needs a response from the author. Publish-Pipeline-Succeeded Retry-1 flag to indicate retried Validation-Completed Validation passed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants