commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kinow <...@git.apache.org>
Subject [GitHub] commons-text issue #70: partial automated migration to assertj
Date Fri, 20 Oct 2017 21:50:21 GMT
Github user kinow commented on the issue:

    https://github.com/apache/commons-text/pull/70
  
    Looks good to me. Given [text] is already on Java 8, I believe it's a good candidate to
start using assertj.
    
    >The script use only the most basic assertj assertions, so the full power is not evident.
    
    Indeed. But I like this approach. We introduce assertj, migrate the old assertions, and
only then we start replacing/simplifying parts of the tests. And new text code can also be
written with assertj.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message