[GitHub] commons-text issue #52: Test: Improved testcase coverage for StrBuilder

2017-06-21 Thread ameyjadiye
Github user ameyjadiye commented on the issue: https://github.com/apache/commons-text/pull/52 Cool, seems clean now :+1: thanks Pascal. Trick will be certainly beneficial in future to me 😉 --- If your project is set up for it, you can reply to this email and have your reply

Re: OSGi Version at Package Level

2017-06-21 Thread Bernd Eckenfels
Hello, Just a pointer, those constraints come in handy when you plan to have OSGi do. the final resolving of actual deployed systems as well. It might not be your cup of coffee but I recommend the modularity talk of Peter Kriens 2016 at the modconf in Darmstadt: https://youtu.be/VwD0pYNOvFQ to

Re: OSGi Version at Package Level

2017-06-21 Thread Bertrand Delacretaz
On Wed, Jun 21, 2017 at 12:08 AM, Simon Spero wrote: > Bundles can specify all sorts of Requirements, including implementations, > and bugfix version ranges...It can be a > little too expressive :-)... Yes, in the OSGi projects where I'm involved we avoid these things as

[GitHub] commons-text issue #52: Test: Improved testcase coverage for StrBuilder

2017-06-21 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/commons-text/pull/52 Looking good! :+1: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes

[GitHub] commons-text issue #52: Test: Improved testcase coverage for StrBuilder

2017-06-21 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue: https://github.com/apache/commons-text/pull/52 Thanks. I was not as easy as I expected (I had to merge). I followed these steps: `git rebase -i head~5` Edit the rebase-todo-to to contain: ``` drop

[GitHub] commons-text issue #52: Test: Improved testcase coverage for StrBuilder

2017-06-21 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-text/pull/52 [![Coverage Status](https://:/builds/12062509/badge)](https://:/builds/12062509) Coverage increased (+0.4%) to 97.317% when pulling **eb25d61ca6bd845815bb30feb2ba0cf9f6e106ff on

[text] Re: CharSequence vs. String (was Re: [GitHub] commons-text pull request #46: TEXT-85:Added CaseUtils class with camel case...)

2017-06-21 Thread Bruno P. Kinoshita
>If a method doesn't intrinsically require a String, then I prefer CharSequence. It's probable that sooner or later something is going to demand a String, but that's not a good reason to be "that guy" :-) I lean towards using CharSequence when that makes sense too (i.e. suggesting we are working

Re: [CLI] Changing the code style

2017-06-21 Thread Gary Gregory
On Fri, Jun 16, 2017 at 1:50 AM, Benedikt Ritter wrote: > Hi, > > CLI unlike other components has a custom code style. While other > components pretty much follow the Sun code guidelines, the CLI code base > uses a style like the Maven project. For me it’s annoying the switch

Fwd: svn commit: r1799506 - in /commons/proper/configuration/trunk/src/site/xdoc: dependencies.xml userguide/howto_hierarchical.xml

2017-06-21 Thread Gary Gregory
Why use an old version of Jackson? Gary -- Forwarded message -- From: Date: Jun 21, 2017 13:01 Subject: svn commit: r1799506 - in /commons/proper/configuration/trunk/src/site/xdoc: dependencies.xml userguide/howto_hierarchical.xml To:

Re: TEXT-77 | String palindrome API

2017-06-21 Thread Gary Gregory
On Jun 21, 2017 12:12, "Emmanuel Bourg" wrote: Le 21/06/2017 à 19:17, Amey Jadiye a écrit : > Though its very trivial to build, by any chance we should have palindrome > method in Apache Commons API ? there is nothing bad having this tool in > our arsenal, preferably in

Re: TEXT-77 | String palindrome API

2017-06-21 Thread Rob Tompkins
> On Jun 21, 2017, at 3:22 PM, Gary Gregory wrote: > > On Jun 21, 2017 12:12, "Emmanuel Bourg" wrote: > >> Le 21/06/2017 à 19:17, Amey Jadiye a écrit : >> >> Though its very trivial to build, by any chance we should have palindrome >> method in

Re: TEXT-77 | String palindrome API

2017-06-21 Thread Emmanuel Bourg
Le 21/06/2017 à 19:17, Amey Jadiye a écrit : > Though its very trivial to build, by any chance we should have palindrome > method in Apache Commons API ? there is nothing bad having this tool in > our arsenal, preferably in StringUtils ? Like sebb I fail to see the use case for this. Checking

Re: TEXT-77 | String palindrome API

2017-06-21 Thread Bruno P. Kinoshita
Palindromes (and other algorithms like De Bruijn graphs, lcs) are normally taught in most introductory bioinformatics courses. Not sure how much palindromes are used though. I know use cases for De Bruijn and lcs. Maybe taking a look at BioJava could help -

Re: TEXT-77 | String palindrome API

2017-06-21 Thread Amey Jadiye
Hi, With little research I found[1] we do have use cases for it like in biology (I think Bruno might be referring the same fields for its application) "Palindrome recognition is important in computational biology. Palindromic structures can frequently be found in proteins and identifying them

[GitHub] commons-text issue #52: Test: Improved testcase coverage for StrBuilder

2017-06-21 Thread chtompki
Github user chtompki commented on the issue: https://github.com/apache/commons-text/pull/52 Thanks Amey. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

[GitHub] commons-text pull request #52: Test: Improved testcase coverage for StrBuild...

2017-06-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-text/pull/52 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

TEXT-77 | String palindrome API

2017-06-21 Thread Amey Jadiye
Hi All, Though its very trivial to build, by any chance we should have palindrome method in Apache Commons API ? there is nothing bad having this tool in our arsenal, preferably in StringUtils ? https://issues.apache.org/jira/browse/TEXT-77 Regards, Amey