You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2020/04/18 04:16:00 UTC
[jira] [Work logged] (COLLECTIONS-759) An issue about the failed
with CI about the checkstyle of NewlineAtEndOfFile.
[ https://issues.apache.org/jira/browse/COLLECTIONS-759?focusedWorklogId=424747&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-424747 ]
ASF GitHub Bot logged work on COLLECTIONS-759:
----------------------------------------------
Author: ASF GitHub Bot
Created on: 18/Apr/20 04:15
Start Date: 18/Apr/20 04:15
Worklog Time Spent: 10m
Work Description: kinow commented on pull request #147: [COLLECTIONS-759]: add newline at end of file, and convert to unix (LF instead of CRLF) fixing checkstyle
URL: https://github.com/apache/commons-collections/pull/147
As per title, fixing the Travis CI build that is broken at the moment due to checkstyle issues. There are some files that were missing newline at end of file (git).
Added the extra line, but after running `mvn` got the error again. Then noticed it was a CRLF vs. LF issue. Ran the files through `dos2unix`, and finally `mvn` command (default build target, same as in Travis) passed.
Now :crossed_fingers: and wait to see what Travis reports.
Bruno
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 424747)
Remaining Estimate: 0h
Time Spent: 10m
> An issue about the failed with CI about the checkstyle of NewlineAtEndOfFile.
> -----------------------------------------------------------------------------
>
> Key: COLLECTIONS-759
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-759
> Project: Commons Collections
> Issue Type: Bug
> Affects Versions: 4.5
> Reporter: Chen Guoping
> Priority: Minor
> Time Spent: 10m
> Remaining Estimate: 0h
>
> commons-collections\src\conf\checkstyle.xml
> <module name="NewlineAtEndOfFile">
> <property name="lineSeparator" value="{color:red}lf{color}" />
> </module>
> [The link of CI|https://travis-ci.org/github/apache/commons-collections/jobs/672844382]
> {code:java}
> [ERROR] src/main/java/org/apache/commons/collections4/ArrayUtils.java:[1] (misc) NewlineAtEndOfFile: Expected line ending for file is LF(\n), but CRLF(\r\n) is detected.
> [ERROR] src/test/java/org/apache/commons/collections4/queue/SynchronizedQueueTest.java:[1] (misc) NewlineAtEndOfFile: Expected line ending for file is LF(\n), but CRLF(\r\n) is detected.
> [ERROR] src/test/java/org/apache/commons/collections4/junit/AbstractAvailableLocalesTest.java:[1] (misc) NewlineAtEndOfFile: Expected line ending for file is LF(\n), but CRLF(\r\n) is detected.
> [ERROR] src/test/java/org/apache/commons/collections4/junit/SetDefaultLocaleTestRule.java:[1] (misc) NewlineAtEndOfFile: Expected line ending for file is LF(\n), but CRLF(\r\n) is detected.
> [ERROR] src/test/java/org/apache/commons/collections4/junit/ObjectToStringComparator.java:[1] (misc) NewlineAtEndOfFile: Expected line ending for file is LF(\n), but CRLF(\r\n) is detected.
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD FAILURE
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 01:41 min
> [INFO] Finished at: 2020-04-09T07:11:16Z
> [INFO] ------------------------------------------------------------------------
> [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.1.1:check (default-cli) on project commons-collections4: You have 5 Checkstyle violations. -> [Help 1]
> {code}
> But I test the checkstyel locally with cmd :
> mvn clean ; mvn checkstyle:checkstyle
> The result is
> Summary
> Files Info Warnings Errors
> 590 0 0 {color:red}590{color}
> Details
> org/apache/commons/collections4/AbstractArrayListTest.java
> Severity Category Rule Message Line
> Error misc {color:red}NewlineAtEndOfFile {color}Expected line ending for file is LF(\n), but CRLF(\r\n) is detected. 1
> Can the checkstyle NewlineAtEndOfFile be warning not error ?
> I don't think the checkstyle would affect the used with the software.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)