-
Notifications
You must be signed in to change notification settings - Fork 8
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
Adapt root and user clients to Y2User #123
Conversation
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.
LGTM. Only minor details.
Pull Request Test Coverage Report for Build 956807824
💛 - Coveralls |
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.
LGTM
✔️ Public Jenkins job #48 successfully finished |
✔️ Internal Jenkins job #31 successfully finished |
Problem
yast2-users code is being adapted to use shadow tools. This module uses some API methods that are going to be dropped in favor of new
Y2Users
classes.Solution
Adapt code to use the new
Y2Users
classes.NOTE: this PR should not be merged yet. It requires yast-users/y2users branch to be merged before.
Test