New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

ArtifactReport.has_* should not rely on the implementation of `with_*` #10

Open

pederhan opened this issue Nov 29, 2022 · 0 comments

Member

pederhan commented Nov 29, 2022 •

edited

Loading

Why create a whole new report and check its artifacts, when we should just be able to do

return any(artifact.has_foo("...") for artifact in self.artifacts)

We already do this with all has_* methods except has_repository and has_severity.

The text was updated successfully, but these errors were encountered:

pederhan added this to Harbor

# for free to join this conversation on GitHub. Already have an account? # to comment