There just were indentation issues.
Fixes: https://pagure.io/389-ds-base/issue/50972
Odd, I didn't see these warnings myself, but nice catch, ACK!
But I need you to change the commit message to use our preferred format:
https://www.port389.org/docs/389ds/contributing.html#getting-the-patch-ready
rebased onto 0e1db960b0a7bbb0debfcd1c2a2c908836ae9cc2
@mreynolds, thank you. Updated.
Looks good, but you should change:
Reviewed by: ???
to
Reviewed by: mreynolds (Thanks!)
:-)
rebased onto f290163114b03b31e84dd5830274afd4e876ba1e
I thought this is the placeholder to be automatically filled up :-)
I wish, so I guess it's a bit confusing, the link I sent is how the commit message should look when you send it out for review. Once reviewed and acked you update the "Reviewed by", with the person or persons, who reviewed it. Anyway thanks again, and I get this cherry-picked to all the branches.
Pull-Request has been merged by mreynolds
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This pull request has been cloned to Github as issue and is available here: - https://github.com/389ds/389-ds-base/issues/4026
If you want to continue to work on the PR, please navigate to the github issue, download the patch from the attachments and file a new pull request.
Thank you for understanding. We apologize for all inconvenience.
Pull-Request has been closed by spichugi
There just were indentation issues.
Fixes: https://pagure.io/389-ds-base/issue/50972