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
Support for SUSE/openSUSE ipaplatform #4532
Conversation
Thanks for your contribution! It's good to finally see SuSE support landing in upstream. The new code uses a slightly different code style than the rest of our code (e.g. single quotes). Could you please run |
The copyright header is
with two empty lines before/after. Could you please squash your commits into a single commit to reduce noise? |
Co-authored-by: Howard Guo <hguo@suse.com> Co-authored-by: Daniel Molkentin <dmolkentin@suse.com> Co-authored-by: Marcus Rückert <darix@nordisch.org>
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, thanks!
LGTM |
master:
|
Upstreaming much of the work done for FreeIPA 4.6 and updated for 4.8 (well, master)
Building the snapshots of this over on https://build.opensuse.org/package/show/home:hellcp/freeipa, although the build service workers time out while building python parts, so not everything might build there until I tweak the spec (or workers) ;)