-
Notifications
You must be signed in to change notification settings - Fork 84
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
Added WiX detection to MSI Parser. #220
Conversation
cb93aa3
to
c7776d6
Compare
c7776d6
to
a7b44c6
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Component Governance down again?
|
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jedieaston, can you file an issue relating to this change? Just want to make sure that every PR has a linked issue for tracking purposes. In the meantime, I'm looking into why the build is failing... |
|
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@jedieaston, sorry for the delay, took some time to figure out what was going on with Component Governance. You should be good to go now :) |
Looks like the changes from main are merged. Ready when you are :D |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Resolves #222.
This PR adds WiX detection to the MSI Parser, inspired by similar changes to YamlCreate.ps1. Unfortuately, since there isn't a silver bullet way to determine if a MSI was created by WiX several things are checked.
Also, it seems to reformat PackageParser.cs for some reason, I suppose there was some style issue that was corrected during commit.
Tested: manually. As always, I'm happy to write unit tests if necessary.
Microsoft Reviewers: Open in CodeFlow