[
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:
[email protected]
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)
