-
Notifications
You must be signed in to change notification settings - Fork 19
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
tests/test_repository.py: Move from dnf repoclosure to zypper + installcheck #423
Conversation
1334ef6
to
59b9579
Compare
59b9579
to
e26f4af
Compare
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.
code wise this change is fine by me. I am not 100% certain that repocheck had no other additional usecases over installcheck though.
from the chat we had yesterday I however understood that we add installcheck into the bci repo publishing pipeline not remove it from bci-tests? what am I missing?
Correct. I had a look but there were two issues:
Changing the code here instead avoided both |
e26f4af
to
ea11324
Compare
5a00dad
to
f7cea0c
Compare
…llcheck Avoids workarounds and dependency on a fedora container.
Avoids workarounds and dependence on a fedora container.