[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-29 Thread ameyjadiye
Github user ameyjadiye commented on the issue: https://github.com/apache/commons-text/pull/62 As per discussion on Dev ML we are undoing deprecation of RandomStringUtils in Lang and discarding this solution. --- -

[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-28 Thread chtompki
Github user chtompki commented on the issue: https://github.com/apache/commons-text/pull/62 @britter, @PascalSchumacher, @garydgregory - thoughts? --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For

[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-27 Thread ameyjadiye
Github user ameyjadiye commented on the issue: https://github.com/apache/commons-text/pull/62 @chtompki , I think all were thinking to deprecate it from lang and so we did. and due to more requests for ```RandomStringUtils``` we came up with the plan to move it also I think since

[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-27 Thread chtompki
Github user chtompki commented on the issue: https://github.com/apache/commons-text/pull/62 As I'm pulling this in this crosses my mind: Wait I'm curious here. Weren't folks leaning towards keeping `RandomStringUtils` in `[lang]`? The thread in question is this one:

[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-27 Thread chtompki
Github user chtompki commented on the issue: https://github.com/apache/commons-text/pull/62 Should be able to look again today. --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional

[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-26 Thread ameyjadiye
Github user ameyjadiye commented on the issue: https://github.com/apache/commons-text/pull/62 Hi @chtompki , do you see more places to improvement or its good now ? --- - To unsubscribe, e-mail:

[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-25 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-text/pull/62 [![Coverage Status](https://coveralls.io/builds/13422531/badge)](https://coveralls.io/builds/13422531) Coverage increased (+0.002%) to 98.238% when pulling

[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-22 Thread chtompki
Github user chtompki commented on the issue: https://github.com/apache/commons-text/pull/62 Sure I'll get to this next. --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands,

[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-22 Thread ameyjadiye
Github user ameyjadiye commented on the issue: https://github.com/apache/commons-text/pull/62 @chtompki , can you take a look if this looks good ? --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For

[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-10 Thread ameyjadiye
Github user ameyjadiye commented on the issue: https://github.com/apache/commons-text/pull/62 @chtompki , can you please take look at this PR ? --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For

[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-10 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-text/pull/62 [![Coverage Status](https://coveralls.io/builds/13202986/badge)](https://coveralls.io/builds/13202986) Coverage increased (+0.0002%) to 98.292% when pulling

[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-10 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-text/pull/62 [![Coverage Status](https://coveralls.io/builds/13202986/badge)](https://coveralls.io/builds/13202986) Coverage increased (+0.0002%) to 98.292% when pulling

[GitHub] commons-text issue #62: TEXT-101: Moved RandomStringUtils to Commons-text

2017-09-06 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-text/pull/62 [![Coverage Status](https://coveralls.io/builds/13157503/badge)](https://coveralls.io/builds/13157503) Coverage decreased (-0.03%) to 98.259% when pulling