-
Notifications
You must be signed in to change notification settings - Fork 162
Refactor for mock plugin #4173
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
Refactor for mock plugin #4173
Conversation
f9fb10e to
7258200
Compare
58006d8 to
53a0fa9
Compare
53a0fa9 to
8802ffc
Compare
c2e40b9 to
31af6b7
Compare
LecrisUT
left a comment
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.
Sorry for not noticing these before (fresh eyes and all that), but there are just a few more refactoring suggestions, but otherwise lgtm.
LecrisUT
left a comment
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.
Awesome, thanks for the work on this 🙂
6b83a88 to
cfadbf0
Compare
|
/packit rebuild-failed |
|
Unrelated failures, merging. |
This among others fixes
Fix the /tests/provision/mock/fedora-rawhide-x86_64/provision-prepare tracebackmentioned in: #4097Pull Request Checklist