> On 15 Jun 2019, at 15:21, sebb <seb...@gmail.com> wrote:
> 
> On Sat, 15 Jun 2019 at 15:14, Gary Gregory <garydgreg...@gmail.com> wrote:
>> 
>> The current master build yields:
>> 
>> [ERROR]   CSVFormatTest.testHashCodeAndWithIgnoreHeaderCase:523
>> [ERROR]   CSVFormatTest.testWithHeaderComments:972
>> 
>> I do not think it is OK to push fixes that break the build.
> 
> Already fixed.

I saw that Sebb fixed the issues that caused my PR to fail. These fails are 
strange as the original PR passed, I modified only comments and formatting 
following Gary’s review and then it failed on openjdk11, 12 and ea despite 
having no changes to code logic. I rebased and it is now passing [1]. 

[1] https://github.com/apache/commons-csv/pull/44 
<https://github.com/apache/commons-csv/pull/44>
> 
>> Gary
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
> 

Reply via email to