On Fri, Feb 12, 2016 at 02:38:45PM +0100, Pavel Březina wrote:
> No.

Running CI should be a part of reviewer's responsibility and we should not
accept patches without a passing CI link -or- a link where the failure is
clearly not related to the diff (maybe a buggy test or an ill CI VM) -or-
where the patch is clearly not covered by our tests (maybe a docs fix).

I ran the CI tests for you:
    http://sssd-ci.duckdns.org/logs/job/37/32/summary.html
but feel free to ask me for instructions next time.

And yes, it's suboptimal that pre-commit CI must be run manually and
requires access to RH VPN, but that's the way it is for now..
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org

Reply via email to