Skip to content
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

Add maintainer for keycloak module #61521

Open
wants to merge 2 commits into
base: devel
from

Conversation

@ndclt
Copy link
Contributor

commented Aug 29, 2019

SUMMARY

Add the author as maintainer of the keycloak module.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

keycloak

ADDITIONAL INFORMATION
@resmo
resmo approved these changes Aug 29, 2019
Copy link
Member

left a comment

LGTM

@ndclt ndclt force-pushed the ndclt:add-maintainer-keycloak-module branch from a493560 to 11ef30d Aug 29, 2019

@ndclt ndclt changed the title add maintainer for keycloak module Add maintainer for keycloak module Aug 29, 2019

@@ -717,7 +717,7 @@ files:
- clustering
- k8s
$module_utils/identity/keycloak/:
maintainers: eikef
maintainers: eikef, ndclt

This comment has been minimized.

Copy link
@gundalow

gundalow Aug 29, 2019

Contributor

Needs to be a space-separated list

Suggested change
maintainers: eikef, ndclt
maintainers: eikef ndclt

This comment has been minimized.

Copy link
@ndclt

ndclt Aug 29, 2019

Author Contributor

Thanks. Corrected in db4d7e9.

@ansibot ansibot removed the needs_rebase label Aug 29, 2019

@gundalow

This comment has been minimized.

Copy link
Contributor

commented Aug 29, 2019

@eikef Are you OK with this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.