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

Installer Detection Enhancements #35735

Merged
merged 4 commits into from
Nov 30, 2021
Merged

Installer Detection Enhancements #35735

merged 4 commits into from
Nov 30, 2021

Conversation

Trenly
Copy link
Contributor

@Trenly Trenly commented Nov 22, 2021

  • Prepare scripts for 1.1.0 manifests
  • Correct Null Checks
  • When Installer URL is the same for multiple entries, skip re-downloading
Microsoft Reviewers: Open in CodeFlow

Trenly and others added 3 commits November 17, 2021 08:16
* Begin update in place

* Condense installer switch logic

* Fix Null values

* Prevent null checking

* Cleanup

* Fix missed variable

* Bump version

* Update function calls

* Fix empty ProductCode

* Confirm Architecture when Automatically Detected

* Remove Debug Line

* Add duplicate url check for Quick update

* Remove line used in debugging
@wingetbot
Copy link
Collaborator

Service Badge  Service Badge  

Tools/YamlCreate.ps1 Show resolved Hide resolved
@ghost ghost added Needs-Author-Feedback This needs a response from the author. 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 Nov 22, 2021
vedantmgoyal9
vedantmgoyal9 previously approved these changes Nov 22, 2021
@vedantmgoyal9
Copy link
Contributor

@denelon

OfficialEsco
OfficialEsco previously approved these changes Nov 22, 2021
@ghost ghost added the Moderator-Approved One of the Moderators has reviewed and approved this PR label Nov 22, 2021
vedantmgoyal9 added a commit to vedantmgoyal9/vedantmgoyal9 that referenced this pull request Nov 23, 2021
* Updated to v1.1.0 manifests
* Correct Null Checks
* When Installer URL is the same for multiple entries, skip re-downloading
@ghost ghost removed Needs-Attention This work item needs to be reviewed by a member of the core team. Moderator-Approved One of the Moderators has reviewed and approved this PR labels Nov 24, 2021
@ghost ghost added the Moderator-Approved One of the Moderators has reviewed and approved this PR label Nov 30, 2021
@denelon denelon merged commit 101954c into microsoft:master Nov 30, 2021
@Trenly Trenly deleted the v2.0.1 branch November 30, 2021 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Moderator-Approved One of the Moderators has reviewed and approved this PR Project-File
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants