stlaz commented on a pull request

"""
Please add ```assert isinstance(dn, DN)``` as in the parent method.

Also, I think maybe it's safer to check for rdns emptiness before you try to 
assign from it.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/68#issuecomment-245878203
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to