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

Fix: Only include unknown packages which exist in source #1972

Merged
merged 1 commit into from Feb 24, 2022

Conversation

Trenly
Copy link
Contributor

@Trenly Trenly commented Feb 24, 2022

This also will only include packages which can be / have been disambiguated

cc @jedieaston


Microsoft Reviewers: Open in CodeFlow

@Trenly Trenly requested a review from as a code owner Feb 24, 2022
@msftbot msftbot bot added the Issue-Bug label Feb 24, 2022
@yao-msft
Copy link
Contributor

@yao-msft yao-msft commented Feb 24, 2022

/azp run

@azure-pipelines
Copy link

@azure-pipelines azure-pipelines bot commented Feb 24, 2022

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@jedieaston jedieaston left a comment

That was it? 🤦‍♂️

thanks for figuring it out :D

@Trenly
Copy link
Contributor Author

@Trenly Trenly commented Feb 24, 2022

That was it? 🤦‍♂️

thanks for figuring it out :D

I think that was it; Seemed to do the check when I tested manually

@denelon denelon merged commit 647b285 into microsoft:master Feb 24, 2022
6 checks passed
@Trenly Trenly deleted the FixUnknownCount branch Feb 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants