Skip to content

Commit 0991214

Browse files
HeartSaVioRkai-chi
authored andcommitted
[MINOR][BUILD] Update all checkstyle dtd to use "https://checkstyle.org"
## What changes were proposed in this pull request? Below build failed with Java checkstyle test, but instead of violation it shows FileNotFound on dtd file. https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/102751/ Looks like the link of dtd file is dead `http://www.puppycrawl.com/dtds/configuration_1_3.dtd`. This patch updates the dtd link to "https://checkstyle.org/dtds/" given checkstyle repository also updated the URL path. checkstyle/checkstyle#5601 ## How was this patch tested? Checked the new links. Closes apache#23887 from HeartSaVioR/java-checkstyle-dtd-change-url. Authored-by: Jungtaek Lim (HeartSaVioR) <[email protected]> Signed-off-by: Marcelo Vanzin <[email protected]> (cherry picked from commit c5de804) Signed-off-by: Marcelo Vanzin <[email protected]>
1 parent 711c036 commit 0991214

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

dev/checkstyle-suppressions.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717

1818
<!DOCTYPE suppressions PUBLIC
1919
"-//Puppy Crawl//DTD Suppressions 1.1//EN"
20-
"http://www.puppycrawl.com/dtds/suppressions_1_1.dtd">
20+
"https://checkstyle.org/dtds/suppressions_1_1.dtd">
2121

2222
<!--
2323

dev/checkstyle.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717

1818
<!DOCTYPE module PUBLIC
1919
"-//Puppy Crawl//DTD Check Configuration 1.3//EN"
20-
"http://www.puppycrawl.com/dtds/configuration_1_3.dtd">
20+
"https://checkstyle.org/dtds/configuration_1_3.dtd">
2121

2222
<!--
2323

0 commit comments

Comments
 (0)