-
-
Notifications
You must be signed in to change notification settings - Fork 377
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/add provided package #1386
Fix/add provided package #1386
Conversation
d382088
to
d6862e4
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1386 +/- ##
=======================================
Coverage 74.86% 74.86%
=======================================
Files 157 157
Lines 18088 18090 +2
=======================================
+ Hits 13541 13543 +2
Misses 3450 3450
Partials 1097 1097 ☔ View full report in Codecov by Sentry. |
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.
Looks good to me 👍🏼
d6862e4
to
c028d5e
Compare
899e013
to
afafbfe
Compare
afafbfe
to
9d05949
Compare
Fixes #1378
Description of the Change
Since ab18d48 list.Search() is also returning provided packages. This is not wanted in all scenarios, for example then adding packages to a repository or in a snapshot pull.
Checklist