URL: https://github.com/freeipa/freeipa/pull/361
Title: #361: This PR implements a number of improvements for our Travis CI:
martbab commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/d86cae7748a8a629c942f1eafc0a0267f2c9611e
URL: https://github.com/freeipa/freeipa/pull/361
Title: #361: This PR implements a number of improvements for our Travis CI:
stlaz commented:
"""
I have no more remarks on this, hopefully final ACK.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/361#issuecomment-270659749
URL: https://github.com/freeipa/freeipa/pull/361
Title: #361: This PR implements a number of improvements for our Travis CI:
stlaz commented:
"""
The change LGTM, ACK, we'll see how it works :)
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/361#issuecomment-270612407
--
URL: https://github.com/freeipa/freeipa/pull/361
Title: #361: This PR implements a number of improvements for our Travis CI:
martbab commented:
"""
@stlaz I have implemented a simple log trimming which keeps only pytest
failures if present. The original behavior is kept as a fallback for the
URL: https://github.com/freeipa/freeipa/pull/361
Title: #361: This PR implements a number of improvements for our Travis CI:
stlaz commented:
"""
@martbab My naive solution is to do something like
```bash
LINE=`grep -n -m 1 $CI_TRAVIS_LOG -e "=== FAILURES ===" | cut -d: -f1`
LINES=`wc -l
URL: https://github.com/freeipa/freeipa/pull/361
Title: #361: This PR implements a number of improvements for our Travis CI:
martbab commented:
"""
@stlaz thanks for an idea, that will actually make the log output much more
useful. I will try to think about it.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/361
Title: #361: This PR implements a number of improvements for our Travis CI:
stlaz commented:
"""
I assume the licence headers did not break the automember tests so this could
be pushed.
Just a brief question: would it be reasonable to get the
URL: https://github.com/freeipa/freeipa/pull/361
Title: #361: This PR implements a number of improvements for our Travis CI:
martbab commented:
"""
Bump for review.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/361#issuecomment-269991386
--
Manage your subscription for
