URL: https://github.com/SSSD/sssd/pull/346
Title: #346: intg: Disable add_remove tests

lslebodn commented:
"""
>I would prefer to have a comment in front of each disabled test saying it is 
>disabled and briefly explaining why (including a reference to the issue 
>perhaps).

Reference to issue is already in commit message and it also contains 
explanation 

Moreover, it is a temporary workaround for unblocking pushing other patches.
Ticket https://pagure.io/SSSD/sssd/issue/3463 is a blocker for 1.15.4 release. 
So this patch will have to be reverted before next release. Adding comment to 
code will not  change situation and therefore such duplication is unnecessary 
for temporary workaround.
Adding a comments directly is a redundant because description in commit
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/346#issuecomment-321728973
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

Reply via email to