Github user PascalSchumacher commented on a diff in the pull request:
https://github.com/apache/commons-text/pull/60#discussion_r130549181
--- Diff: src/test/java/org/apache/commons/text/StringEscapeUtilsTest.java
---
@@ -592,4 +593,43 @@ public void testUnscapeXSI() {
Github user TheRealHaui commented on a diff in the pull request:
https://github.com/apache/commons-text/pull/60#discussion_r130609309
--- Diff: src/test/java/org/apache/commons/text/StringEscapeUtilsTest.java
---
@@ -592,4 +593,43 @@ public void testUnscapeXSI() {
asse
Github user TheRealHaui commented on the issue:
https://github.com/apache/commons-text/pull/60
Rechecked for blank lines and removed them.
---
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
Github user TheRealHaui commented on the issue:
https://github.com/apache/commons-imaging/pull/30
Checks failed due to Travis JDK crashing not due to problems in code.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user coveralls commented on the issue:
https://github.com/apache/commons-text/pull/60
[![Coverage
Status](https://coveralls.io/builds/12639540/badge)](https://coveralls.io/builds/12639540)
Coverage increased (+0.3%) to 98.489% when pulling
**fd33b1cbbc7e1188ebe2b1
Hi all
with the binding +1s by Oliver Heger, Gary Gregory and myself (cast
implicitly) as well as the +1 by Amey Jadiye this vote has passed.
I'll publish the artifacts and give the mirrors a little time to catch
up before announcing the release.
Many thanks to those who reviewed the release
Github user PascalSchumacher commented on a diff in the pull request:
https://github.com/apache/commons-text/pull/60#discussion_r130633380
--- Diff: src/test/java/org/apache/commons/text/StringEscapeUtilsTest.java
---
@@ -592,4 +593,43 @@ public void testUnscapeXSI() {
Github user coveralls commented on the issue:
https://github.com/apache/commons-text/pull/57
[![Coverage
Status](https://coveralls.io/builds/12640859/badge)](https://coveralls.io/builds/12640859)
Coverage decreased (-0.2%) to 98.021% when pulling
**fb6d5935451397c561bd52
Github user arunvinudss commented on the issue:
https://github.com/apache/commons-text/pull/57
@chtompki Please review and merge.
---
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
Github user chtompki commented on the issue:
https://github.com/apache/commons-text/pull/57
Will get to this today.
---
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 wi
Github user asfgit closed the pull request at:
https://github.com/apache/commons-text/pull/60
---
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 is
Github user PascalSchumacher commented on the issue:
https://github.com/apache/commons-text/pull/60
Thanks! :+1:
By the way: Feel free to raise a jira issue for anything you think that
maybe be a bug (e.g. NullPointerExceptions).
---
If your project is set up for it, you c
Github user asfgit closed the pull request at:
https://github.com/apache/commons-text/pull/57
---
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 is
It would be nice to add checkstyle and findbug as well ?
Regards,
Amey
-- Forwarded message --
From:
Date: Tue, Aug 1, 2017 at 10:57 PM
Subject: [2/2] commons-cli git commit: add default mvn default (clean,
test, clirr, rat and javadoc) and run it on travis
To: comm...@commons.ap
Imho yes, but to make this possible existing violations have to be
analyzed and then either fixed or ignored.
Am 01.08.2017 um 19:37 schrieb Amey Jadiye:
It would be nice to add checkstyle and findbug as well ?
Regards,
Amey
-- Forwarded message --
From:
Date: Tue, Aug 1, 201
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
The Apache Commons Team is pleased to announce the release of Apache
Commons Email 1.5.
This is a major and security bugfix release which adds some new
features and fixes several bugs present in the 1.4 release. All
current users are encouraged to upg
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
CVE-2017-9801: Apache Commons Email SMTP header injection vulnerabilty
Severity: low
Vendor:
The Apache Software Foundation
Versions Affected:
Apache Commons Email 1.0 to 1.4.
Description:
When a call-site passes a subject for an email that contain
GitHub user ameyjadiye opened a pull request:
https://github.com/apache/commons-cli/pull/16
added findbug and checkstyle support while building
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ameyjadiye/commons-cli master
Altern
Just checked commons-cli pom.xml and found checkstyle and findbug support
is already present in reporting section, just copied same setting in build
section and we are good.
https://github.com/apache/commons-cli/pull/16
You are welcome :-)
Regards,
Amey
On Tue, Aug 1, 2017 at 11:26 PM, Pascal
Github user coveralls commented on the issue:
https://github.com/apache/commons-cli/pull/16
[![Coverage
Status](https://coveralls.io/builds/12646111/badge)](https://coveralls.io/builds/12646111)
Coverage remained the same at 96.287% when pulling
**d6dcadb6d7c8a25996a0e71
Github user coveralls commented on the issue:
https://github.com/apache/commons-cli/pull/16
[![Coverage
Status](https://coveralls.io/builds/12646111/badge)](https://coveralls.io/builds/12646111)
Coverage remained the same at 96.287% when pulling
**d6dcadb6d7c8a25996a0e71
Github user TheRealHaui commented on the issue:
https://github.com/apache/commons-text/pull/60
You're welcome!
Thought it would be preferred by comments.
However, I will try to keep in mind.
And be cautious what you wish. ;-)
---
If your project is set up for it, you can
GitHub user ameyjadiye opened a pull request:
https://github.com/apache/commons-text/pull/61
Added flexible Travis-ci build script.
This enables ua to use any jdk against our code and not keeps us dependency
on Travis to provide jdk
You can merge this pull request into a Git repo
Github user chtompki commented on the issue:
https://github.com/apache/commons-text/pull/61
I'll get to this soon.
---
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 wis
Github user PascalSchumacher commented on the issue:
https://github.com/apache/commons-text/pull/61
In my opinion this is overkill. I don't think commons-text needs to be
tested against the eclipse java compiler and early access version of java 8 and
9. The script looks difficult to d
25 matches
Mail list logo