added filtering by artifact type with regex

[wangxiaoxuan273]

Signed-off-by: wangxiaoxuan273 [email protected]

Resolves issue #201

[wangxiaoxuan273]

Further clarification needed: do we need to support doing both inclusion and exclusion at the same time (intersection) ? Right now it is either inclusion or exclusion.

[wangxiaoxuan273]

To generalize the above question: should this filtering feature modify the existing FindPredecessors, or creating new FindPredecessors? My current implementation is creating a new FindPredecessors function that either includes or excludes on artifact type with a regex. What should this filtering feature do with existing FindPredecessors ? @shizhMSFT

[shizhMSFT]

To generalize the above question: should this filtering feature modify the existing FindPredecessors, or creating new FindPredecessors? My current implementation is creating a new FindPredecessors function that either includes or excludes on artifact type with a regex. What should this filtering feature do with existing FindPredecessors ? @shizhMSFT

We should invoke the existing FindPredecessors first, and then do filtering.

[codecov-commenter]

Codecov Report

Merging #258 (bd5782c) into main (e24e247) will increase coverage by 0.05%.
The diff coverage is 73.68%.

@@            Coverage Diff             @@
##             main     #258      +/-   ##
==========================================
+ Coverage   70.68%   70.73%   +0.05%     
==========================================
  Files          36       36              
  Lines        3217     3274      +57     
==========================================
+ Hits         2274     2316      +42     
- Misses        700      712      +12     
- Partials      243      246       +3     

Impacted Files	Coverage Δ
registry/repository.go	0.00% <ø> (ø)
extendedcopy.go	70.39% <73.68%> (+1.97%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[shizhMSFT]

There is a performance issue on obtaining the artifact type. We should avoid unnecessary calls when Referrer API is available.

[shizhMSFT]

LGTM