#50973 Fix cockpit plugin build
Closed by spichugi. Opened by slev.
slev/389-ds-base fix_cockpit_build  into  master

Download 50973.patch

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.

@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 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

Metadata