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/add provided package #1386

Merged
merged 6 commits into from
Nov 8, 2024
Merged

Fix/add provided package #1386

merged 6 commits into from
Nov 8, 2024

Conversation

neolynx
Copy link
Member

@neolynx neolynx commented Nov 4, 2024

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

  • unit-test added (if change is algorithm)
  • functional test added/updated (if change is functional)

@neolynx neolynx requested a review from 5hir0kur0 November 4, 2024 15:08
@neolynx neolynx force-pushed the fix/add-provided-package branch from d382088 to d6862e4 Compare November 4, 2024 15:43
Copy link

codecov bot commented Nov 4, 2024

Codecov Report

Attention: Patch coverage is 91.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 74.86%. Comparing base (7436454) to head (9d05949).
Report is 7 commits behind head on master.

Files with missing lines Patch % Lines
api/snapshot.go 0.00% 1 Missing ⚠️
deb/package_collection.go 0.00% 1 Missing ⚠️
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.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@5hir0kur0 5hir0kur0 left a 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 👍🏼

@neolynx neolynx force-pushed the fix/add-provided-package branch from d6862e4 to c028d5e Compare November 7, 2024 09:00
@neolynx neolynx requested a review from a team November 7, 2024 16:31
@neolynx neolynx self-assigned this Nov 7, 2024
@neolynx neolynx added the needs review Ready for review & merge label Nov 7, 2024
@neolynx neolynx force-pushed the fix/add-provided-package branch 6 times, most recently from 899e013 to afafbfe Compare November 8, 2024 09:19
@neolynx neolynx force-pushed the fix/add-provided-package branch from afafbfe to 9d05949 Compare November 8, 2024 09:20
@neolynx neolynx merged commit 6647f6d into master Nov 8, 2024
46 checks passed
@neolynx neolynx deleted the fix/add-provided-package branch November 8, 2024 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs review Ready for review & merge
Projects
None yet
3 participants