URL: https://github.com/SSSD/sssd/pull/139
Title: #139: Initial revision of sssd pytest framework

fidencio commented:
"""
I've been able to test the patches with some documentation sent by @mrniranjan. 
As this documentation is related to provisioning downstream infra, it's not 
worth (nor allowed) to have it published upstream.

The PR works as expected but I'd like to ask for a few changes:
- Please, if it makes sense for you, squash 
https://github.com/fidencio/sssd/commit/6db7bcfd06c36c687f6e7cd715638267643818e8
 into your patch
- Please, remove the generated html docs

Once it's done, please, re-submit the PR and let's be sure our internal CI 
passes and I'll give it my Ack!

Thanks a lot for your work and for your patience!
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/139#issuecomment-344557055
_______________________________________________
sssd-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to