Merge pull request #4 from AlexisJehan/3-coments-are-not-allowed-beca…

…use-of-artifactfilterparseexception

Fix the UTF-8 with BOM encoding compatibility for ignore files

src/main/java/com/github/alexisjehan/mvncheck/core/component/filter/artifact/parser/ArtifactFilterParser.java

@@ -98,6 +98,10 @@ public static ArtifactFilter parse(final Path ignoreFile) throws IOException {
 		Ensure.notNullAndFile("ignoreFile", ignoreFile);
 		logger.info("Parsing the {} ignore file", () -> ToString.toString(ignoreFile));
 		try (var reader = Readers.of(ignoreFile)) {
+			reader.mark(1);
+			if ('\ufeff' != reader.read()) { // UTF-8 BOM character
+				reader.reset();
+			}
 			return parse(reader);
 		} catch (final ArtifactFilterParseException e) {
 			throw e.with(ignoreFile);

src/test/java/com/github/alexisjehan/mvncheck/core/component/filter/artifact/parser/ArtifactFilterParserTest.java

@@ -54,6 +54,9 @@ void testParsePath() {
 		assertThatNoException().isThrownBy(
 				() -> ArtifactFilterParser.parse(Path.of("src", "test", "resources", ".mvnchk-ignore_foo"))
 		);
+		assertThatNoException().isThrownBy(
+				() -> ArtifactFilterParser.parse(Path.of("src", "test", "resources", ".mvnchk-ignore_bom"))
+		);
 	}
 
 	@Test

src/test/resources/.mvnchk-ignore_bom

@@ -0,0 +1 @@
+# comment