kwin commented on pull request #79: URL: https://github.com/apache/maven-enforcer/pull/79#issuecomment-706007387
> The current enforcer RequireFileChecksum checks file checksum Agree, but that includes both binary and text files. There is no hint in https://maven.apache.org/enforcer/enforcer-rules/requireFileChecksum.html that this should only be used for binaries. > So, I'd propose we introduce a new RequireTextFileChecksum enforcer to make it clear. I am fine with that as well, but honestly I don't see any benefit here. As both need configuration per file, one can just as well add another configuration flag (`normalizeLineSeparatorTo`). Why do you prefer two different rule names here? ---------------------------------------------------------------- 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]
