[GitHub] commons-cli pull request #18: .travis.yml: install open-jdk6, as it was remo...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/commons-cli/pull/18


---

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

[GitHub] commons-cli pull request #18: .travis.yml: install open-jdk6, as it was remo...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/commons-cli/pull/18


---

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

[GitHub] commons-cli pull request #17: Tweak Javadoc spelling

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/commons-cli/pull/17


---

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

[GitHub] commons-cli pull request #17: Tweak Javadoc spelling

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/commons-cli/pull/17


---

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

[GitHub] commons-cli pull request #16: added findbug and checkstyle support while bui...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/commons-cli/pull/16


---

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

[GitHub] commons-cli pull request #14: Added BugCLI260 JUnit Test

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/commons-cli/pull/14


---

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

[GitHub] commons-cli pull request #16: added findbug and checkstyle support while bui...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/commons-cli/pull/16


---

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

[GitHub] commons-cli pull request #14: Added BugCLI260 JUnit Test

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/commons-cli/pull/14


---

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

[GitHub] commons-text pull request #63: small-code-quality-improvements

[TheRealHaui]

Github user TheRealHaui commented on a diff in the pull request:

https://github.com/apache/commons-text/pull/63#discussion_r140524746
  
--- Diff: src/main/java/org/apache/commons/text/FormattableUtils.java ---
@@ -46,6 +46,7 @@
  * This constructor is public to permit tools that require a 
JavaBean
  * instance to operate.
  */
+@SuppressWarnings("squid:S1118")
--- End diff --

Well, works with Sonaqube and Intellij IDEA.
Therefore I assume Eclipse and Netbeans recognizing it too.

I guess you want it to removed.
True?


---

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

[GitHub] commons-text pull request #63: small-code-quality-improvements

[TheRealHaui]

Github user TheRealHaui commented on a diff in the pull request:

https://github.com/apache/commons-text/pull/63#discussion_r140525860
  
--- Diff: src/main/java/org/apache/commons/text/WordUtils.java ---
@@ -614,13 +615,11 @@ public static String swapCase(final String str) {
 for (int index = 0; index < strLen;) {
 final int oldCodepoint = str.codePointAt(index);
 final int newCodePoint;
-if (Character.isUpperCase(oldCodepoint)) {
+if (Character.isUpperCase(oldCodepoint) || 
Character.isTitleCase(oldCodepoint) ) {
--- End diff --

The corresponding lines as the whole method are already completely covered 
by existing Unit Tests.

![selection_050](https://user-images.githubusercontent.com/6312834/30752358-5a0d0094-9fbc-11e7-9ada-822cb344cefb.png)



---

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

[GitHub] commons-text pull request #63: small-code-quality-improvements

[TheRealHaui]

Github user TheRealHaui commented on a diff in the pull request:

https://github.com/apache/commons-text/pull/63#discussion_r140526276
  
--- Diff: src/main/java/org/apache/commons/text/WordUtils.java ---
@@ -614,13 +615,11 @@ public static String swapCase(final String str) {
 for (int index = 0; index < strLen;) {
 final int oldCodepoint = str.codePointAt(index);
 final int newCodePoint;
-if (Character.isUpperCase(oldCodepoint)) {
+if (Character.isUpperCase(oldCodepoint) || 
Character.isTitleCase(oldCodepoint) ) {
 newCodePoint = Character.toLowerCase(oldCodepoint);
 whitespace = false;
-} else if (Character.isTitleCase(oldCodepoint)) {
-newCodePoint = Character.toLowerCase(oldCodepoint);
-whitespace = false;
-} else if (Character.isLowerCase(oldCodepoint)) {
+}
+else if (Character.isLowerCase(oldCodepoint)) {
--- End diff --

No, you're right.
My mistake.

Corrected and commited the correction.


---

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

[GitHub] commons-text pull request #63: small-code-quality-improvements

[PascalSchumacher]

Github user PascalSchumacher commented on a diff in the pull request:

https://github.com/apache/commons-text/pull/63#discussion_r140526804
  
--- Diff: src/main/java/org/apache/commons/text/FormattableUtils.java ---
@@ -46,6 +46,7 @@
  * This constructor is public to permit tools that require a 
JavaBean
  * instance to operate.
  */
+@SuppressWarnings("squid:S1118")
--- End diff --

-1 to this

Eclipse was not displaying any warning before, but with this change it 
displays this warning: `Unsupported @SuppressWarnings("squid:S1118")`


---

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

[GitHub] commons-text pull request #63: small-code-quality-improvements

[TheRealHaui]

Github user TheRealHaui commented on a diff in the pull request:

https://github.com/apache/commons-text/pull/63#discussion_r140528491
  
--- Diff: src/main/java/org/apache/commons/text/FormattableUtils.java ---
@@ -46,6 +46,7 @@
  * This constructor is public to permit tools that require a 
JavaBean
  * instance to operate.
  */
+@SuppressWarnings("squid:S1118")
--- End diff --

Removed SuppressWarnings anootations completely.


---

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

[GitHub] commons-text pull request #63: small-code-quality-improvements

[chtompki]

Github user chtompki commented on a diff in the pull request:

https://github.com/apache/commons-text/pull/63#discussion_r140545049
  
--- Diff: src/main/java/org/apache/commons/text/WordUtils.java ---
@@ -614,10 +614,7 @@ public static String swapCase(final String str) {
 for (int index = 0; index < strLen;) {
 final int oldCodepoint = str.codePointAt(index);
 final int newCodePoint;
-if (Character.isUpperCase(oldCodepoint)) {
-newCodePoint = Character.toLowerCase(oldCodepoint);
-whitespace = false;
-} else if (Character.isTitleCase(oldCodepoint)) {
+if (Character.isUpperCase(oldCodepoint) || 
Character.isTitleCase(oldCodepoint) ) {
--- End diff --

Checkstyle want's to see this as

```java
if (Character.isUpperCase(oldCodepoint) || 
Character.isTitleCase(oldCodepoint)) {
```


---

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

[GitHub] commons-text pull request #63: small-code-quality-improvements

[TheRealHaui]

Github user TheRealHaui commented on a diff in the pull request:

https://github.com/apache/commons-text/pull/63#discussion_r140548121
  
--- Diff: src/main/java/org/apache/commons/text/WordUtils.java ---
@@ -614,10 +614,7 @@ public static String swapCase(final String str) {
 for (int index = 0; index < strLen;) {
 final int oldCodepoint = str.codePointAt(index);
 final int newCodePoint;
-if (Character.isUpperCase(oldCodepoint)) {
-newCodePoint = Character.toLowerCase(oldCodepoint);
-whitespace = false;
-} else if (Character.isTitleCase(oldCodepoint)) {
+if (Character.isUpperCase(oldCodepoint) || 
Character.isTitleCase(oldCodepoint) ) {
--- End diff --

Removed white space.


---

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

Re: [text] license header

[Gary Gregory]

Good catch and thank you.

I wish we could set up Maven to run the RAT check and fail a local build if
a header is missing.

Gary


On Fri, Sep 22, 2017 at 11:24 AM,  wrote:

> Repository: commons-text
> Updated Branches:
>   refs/heads/master 3292f74df -> 8cdbbd34b
>
>
> license header
>
>
> Project: http://git-wip-us.apache.org/repos/asf/commons-text/repo
> Commit: http://git-wip-us.apache.org/repos/asf/commons-text/commit/
> 8cdbbd34
> Tree: http://git-wip-us.apache.org/repos/asf/commons-text/tree/8cdbbd34
> Diff: http://git-wip-us.apache.org/repos/asf/commons-text/diff/8cdbbd34
>
> Branch: refs/heads/master
> Commit: 8cdbbd34be793c828fb2000a3d75661aacae7dfc
> Parents: 3292f74
> Author: Rob Tompkins 
> Authored: Fri Sep 22 13:24:00 2017 -0400
> Committer: Rob Tompkins 
> Committed: Fri Sep 22 13:24:00 2017 -0400
>
> --
>  .../resources/testResourceBundleLookup.properties| 15 +++
>  1 file changed, 15 insertions(+)
> --
>
>
> http://git-wip-us.apache.org/repos/asf/commons-text/blob/
> 8cdbbd34/src/test/resources/testResourceBundleLookup.properties
> --
> diff --git a/src/test/resources/testResourceBundleLookup.properties
> b/src/test/resources/testResourceBundleLookup.properties
> index f1394e7..ea39746 100644
> --- a/src/test/resources/testResourceBundleLookup.properties
> +++ b/src/test/resources/testResourceBundleLookup.properties
> @@ -1,2 +1,17 @@
> +# Licensed to the Apache Software Foundation (ASF) under one or more
> +# contributor license agreements.  See the NOTICE file distributed with
> +# this work for additional information regarding copyright ownership.
> +# The ASF licenses this file to You under the Apache License, Version 2.0
> +# (the "License"); you may not use this file except in compliance with
> +# the License.  You may obtain a copy of the License at
> +#
> +#  http://www.apache.org/licenses/LICENSE-2.0
> +#
> +# Unless required by applicable law or agreed to in writing, software
> +# distributed under the License is distributed on an "AS IS" BASIS,
> +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
> +# See the License for the specific language governing permissions and
> +# limitations under the License.
> +
>  key = value
>  number = 2
>
>

[GitHub] commons-text issue #63: small-code-quality-improvements

[chtompki]

Github user chtompki commented on the issue:

https://github.com/apache/commons-text/pull/63
  
That last build failure wasn't on youjust fixed it in the master branch.


---

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

Re: [text] license header

[Gary Gregory]

Hm, I guess we could add 'apache-rat:check' and 'clirr:check' to the Travis
and Jenkins builds.

Gary

On Fri, Sep 22, 2017 at 11:26 AM, Gary Gregory 
wrote:

> Good catch and thank you.
>
> I wish we could set up Maven to run the RAT check and fail a local build
> if a header is missing.
>
> Gary
>
>
> On Fri, Sep 22, 2017 at 11:24 AM,  wrote:
>
>> Repository: commons-text
>> Updated Branches:
>>   refs/heads/master 3292f74df -> 8cdbbd34b
>>
>>
>> license header
>>
>>
>> Project: http://git-wip-us.apache.org/repos/asf/commons-text/repo
>> Commit: http://git-wip-us.apache.org/repos/asf/commons-text/commit/8
>> cdbbd34
>> Tree: http://git-wip-us.apache.org/repos/asf/commons-text/tree/8cdbbd34
>> Diff: http://git-wip-us.apache.org/repos/asf/commons-text/diff/8cdbbd34
>>
>> Branch: refs/heads/master
>> Commit: 8cdbbd34be793c828fb2000a3d75661aacae7dfc
>> Parents: 3292f74
>> Author: Rob Tompkins 
>> Authored: Fri Sep 22 13:24:00 2017 -0400
>> Committer: Rob Tompkins 
>> Committed: Fri Sep 22 13:24:00 2017 -0400
>>
>> --
>>  .../resources/testResourceBundleLookup.properties| 15
>> +++
>>  1 file changed, 15 insertions(+)
>> --
>>
>>
>> http://git-wip-us.apache.org/repos/asf/commons-text/blob/8cd
>> bbd34/src/test/resources/testResourceBundleLookup.properties
>> --
>> diff --git a/src/test/resources/testResourceBundleLookup.properties
>> b/src/test/resources/testResourceBundleLookup.properties
>> index f1394e7..ea39746 100644
>> --- a/src/test/resources/testResourceBundleLookup.properties
>> +++ b/src/test/resources/testResourceBundleLookup.properties
>> @@ -1,2 +1,17 @@
>> +# Licensed to the Apache Software Foundation (ASF) under one or more
>> +# contributor license agreements.  See the NOTICE file distributed with
>> +# this work for additional information regarding copyright ownership.
>> +# The ASF licenses this file to You under the Apache License, Version 2.0
>> +# (the "License"); you may not use this file except in compliance with
>> +# the License.  You may obtain a copy of the License at
>> +#
>> +#  http://www.apache.org/licenses/LICENSE-2.0
>> +#
>> +# Unless required by applicable law or agreed to in writing, software
>> +# distributed under the License is distributed on an "AS IS" BASIS,
>> +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
>> implied.
>> +# See the License for the specific language governing permissions and
>> +# limitations under the License.
>> +
>>  key = value
>>  number = 2
>>
>>
>

Re: [text] license header

[Amey Jadiye]

Its already there [1] and fails if something is not correct in *main* but I
guess it doesn't check anything for test resources and test java files.
thats why build was passing till date.


[1]. https://github.com/apache/commons-text/blob/master/pom.xml

Regards,
Amey

On Fri, Sep 22, 2017 at 10:58 PM, Gary Gregory 
wrote:

> Hm, I guess we could add 'apache-rat:check' and 'clirr:check' to the Travis
> and Jenkins builds.
>
> Gary
>
> On Fri, Sep 22, 2017 at 11:26 AM, Gary Gregory 
> wrote:
>
> > Good catch and thank you.
> >
> > I wish we could set up Maven to run the RAT check and fail a local build
> > if a header is missing.
> >
> > Gary
> >
> >
> > On Fri, Sep 22, 2017 at 11:24 AM,  wrote:
> >
> >> Repository: commons-text
> >> Updated Branches:
> >>   refs/heads/master 3292f74df -> 8cdbbd34b
> >>
> >>
> >> license header
> >>
> >>
> >> Project: http://git-wip-us.apache.org/repos/asf/commons-text/repo
> >> Commit: http://git-wip-us.apache.org/repos/asf/commons-text/commit/8
> >> cdbbd34
> >> Tree: http://git-wip-us.apache.org/repos/asf/commons-text/tree/8cdbbd34
> >> Diff: http://git-wip-us.apache.org/repos/asf/commons-text/diff/8cdbbd34
> >>
> >> Branch: refs/heads/master
> >> Commit: 8cdbbd34be793c828fb2000a3d75661aacae7dfc
> >> Parents: 3292f74
> >> Author: Rob Tompkins 
> >> Authored: Fri Sep 22 13:24:00 2017 -0400
> >> Committer: Rob Tompkins 
> >> Committed: Fri Sep 22 13:24:00 2017 -0400
> >>
> >> --
> >>  .../resources/testResourceBundleLookup.properties| 15
> >> +++
> >>  1 file changed, 15 insertions(+)
> >> --
> >>
> >>
> >> http://git-wip-us.apache.org/repos/asf/commons-text/blob/8cd
> >> bbd34/src/test/resources/testResourceBundleLookup.properties
> >> --
> >> diff --git a/src/test/resources/testResourceBundleLookup.properties
> >> b/src/test/resources/testResourceBundleLookup.properties
> >> index f1394e7..ea39746 100644
> >> --- a/src/test/resources/testResourceBundleLookup.properties
> >> +++ b/src/test/resources/testResourceBundleLookup.properties
> >> @@ -1,2 +1,17 @@
> >> +# Licensed to the Apache Software Foundation (ASF) under one or more
> >> +# contributor license agreements.  See the NOTICE file distributed with
> >> +# this work for additional information regarding copyright ownership.
> >> +# The ASF licenses this file to You under the Apache License, Version
> 2.0
> >> +# (the "License"); you may not use this file except in compliance with
> >> +# the License.  You may obtain a copy of the License at
> >> +#
> >> +#  http://www.apache.org/licenses/LICENSE-2.0
> >> +#
> >> +# Unless required by applicable law or agreed to in writing, software
> >> +# distributed under the License is distributed on an "AS IS" BASIS,
> >> +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
> >> implied.
> >> +# See the License for the specific language governing permissions and
> >> +# limitations under the License.
> >> +
> >>  key = value
> >>  number = 2
> >>
> >>
> >
>



-- 

-

To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org

For additional commands, e-mail: dev-h...@commons.apache.org

Re: [text] license header

[Amey Jadiye]

Ah, my bad actually after your commit for TEXT-102 Travis was failed [1].
so seems we are good here.

[1] https://travis-ci.org/apache/commons-text/builds/278697845

Regards,
Amey

On Fri, Sep 22, 2017 at 11:41 PM, Amey Jadiye  wrote:

> Its already there [1] and fails if something is not correct in *main* but
> I guess it doesn't check anything for test resources and test java files.
> thats why build was passing till date.
>
>
> [1]. https://github.com/apache/commons-text/blob/master/pom.xml
>
> Regards,
> Amey
>
> On Fri, Sep 22, 2017 at 10:58 PM, Gary Gregory 
> wrote:
>
>> Hm, I guess we could add 'apache-rat:check' and 'clirr:check' to the
>> Travis
>> and Jenkins builds.
>>
>> Gary
>>
>> On Fri, Sep 22, 2017 at 11:26 AM, Gary Gregory 
>> wrote:
>>
>> > Good catch and thank you.
>> >
>> > I wish we could set up Maven to run the RAT check and fail a local build
>> > if a header is missing.
>> >
>> > Gary
>> >
>> >
>> > On Fri, Sep 22, 2017 at 11:24 AM,  wrote:
>> >
>> >> Repository: commons-text
>> >> Updated Branches:
>> >>   refs/heads/master 3292f74df -> 8cdbbd34b
>> >>
>> >>
>> >> license header
>> >>
>> >>
>> >> Project: http://git-wip-us.apache.org/repos/asf/commons-text/repo
>> >> Commit: http://git-wip-us.apache.org/repos/asf/commons-text/commit/8
>> >> cdbbd34
>> >> Tree: http://git-wip-us.apache.org/repos/asf/commons-text/tree/8cd
>> bbd34
>> >> Diff: http://git-wip-us.apache.org/repos/asf/commons-text/diff/8cd
>> bbd34
>> >>
>> >> Branch: refs/heads/master
>> >> Commit: 8cdbbd34be793c828fb2000a3d75661aacae7dfc
>> >> Parents: 3292f74
>> >> Author: Rob Tompkins 
>> >> Authored: Fri Sep 22 13:24:00 2017 -0400
>> >> Committer: Rob Tompkins 
>> >> Committed: Fri Sep 22 13:24:00 2017 -0400
>> >>
>> >> --
>> >>  .../resources/testResourceBundleLookup.properties| 15
>> >> +++
>> >>  1 file changed, 15 insertions(+)
>> >> --
>> >>
>> >>
>> >> http://git-wip-us.apache.org/repos/asf/commons-text/blob/8cd
>> >> bbd34/src/test/resources/testResourceBundleLookup.properties
>> >> --
>> >> diff --git a/src/test/resources/testResourceBundleLookup.properties
>> >> b/src/test/resources/testResourceBundleLookup.properties
>> >> index f1394e7..ea39746 100644
>> >> --- a/src/test/resources/testResourceBundleLookup.properties
>> >> +++ b/src/test/resources/testResourceBundleLookup.properties
>> >> @@ -1,2 +1,17 @@
>> >> +# Licensed to the Apache Software Foundation (ASF) under one or more
>> >> +# contributor license agreements.  See the NOTICE file distributed
>> with
>> >> +# this work for additional information regarding copyright ownership.
>> >> +# The ASF licenses this file to You under the Apache License, Version
>> 2.0
>> >> +# (the "License"); you may not use this file except in compliance with
>> >> +# the License.  You may obtain a copy of the License at
>> >> +#
>> >> +#  http://www.apache.org/licenses/LICENSE-2.0
>> >> +#
>> >> +# Unless required by applicable law or agreed to in writing, software
>> >> +# distributed under the License is distributed on an "AS IS" BASIS,
>> >> +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
>> >> implied.
>> >> +# See the License for the specific language governing permissions and
>> >> +# limitations under the License.
>> >> +
>> >>  key = value
>> >>  number = 2
>> >>
>> >>
>> >
>>
>
>
>
> --
>
> -
>
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
>
> For additional commands, e-mail: dev-h...@commons.apache.org
>



-- 

-

To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org

For additional commands, e-mail: dev-h...@commons.apache.org

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

[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 additional commands, e-mail: dev-h...@commons.apache.org

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

[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, e-mail: dev-h...@commons.apache.org

[GitHub] commons-cli issue #16: added findbug and checkstyle support while building

[ameyjadiye]

Github user ameyjadiye commented on the issue:

https://github.com/apache/commons-cli/pull/16
  
@chtompki , is this not accepted ?


---

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

[GitHub] commons-cli issue #16: added findbug and checkstyle support while building

[ameyjadiye]

Github user ameyjadiye commented on the issue:

https://github.com/apache/commons-cli/pull/16
  
@chtompki , is this not accepted ?


---

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

[GitHub] commons-cli issue #16: added findbug and checkstyle support while building

[chtompki]

Github user chtompki commented on the issue:

https://github.com/apache/commons-cli/pull/16
  
Oh pardon. I misread. I’ll re-run through it. 


---

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

[GitHub] commons-cli issue #16: added findbug and checkstyle support while building

[chtompki]

Github user chtompki commented on the issue:

https://github.com/apache/commons-cli/pull/16
  
Oh pardon. I misread. I’ll re-run through it. 


---

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

Re: [apache/commons-text] [TEXT-102] StrLookup.resourceBundleLookup(ResourceBundle). Make new (0b06169)

[Gary Gregory]

Fixed in git master.

Gary

On Fri, Sep 22, 2017 at 3:35 PM, Pascal Schumacher  wrote:

> Builds on travis fail:
>
> [INFO] There are 6 checkstyle errors.
> [ERROR] StrLookup.java[119] (regexp) RegexpSingleline: Line has trailing 
> spaces.
> [ERROR] StrLookup.java[125] (regexp) RegexpSingleline: Line has trailing 
> spaces.
> [ERROR] StrLookup.java[181:9] (javadoc) JavadocVariable: Missing a Javadoc 
> comment.
> [ERROR] StrLookup.java[183:9] (javadoc) JavadocMethod: Missing a Javadoc 
> comment.
> [ERROR] StrLookup.java[194] (regexp) RegexpSingleline: Line has trailing 
> spaces.
> [ERROR] StrLookup.java[199] (regexp) RegexpSingleline: Line has trailing 
> spaces.
>
> see e.g.: https://travis-ci.org/apache/commons-text/jobs/278728858
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> ,
> or mute the thread
> 
> .
>