Skip to content

Commit ccfc7d6

Browse files
authored
[CI] Ensure that htmltest-config warnings fail GH check (open-telemetry#5612)
1 parent cd73780 commit ccfc7d6

File tree

4 files changed

+6
-7
lines changed

4 files changed

+6
-7
lines changed

.warnings-skip-list.txt

+1
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
11
_filename-error
2+
The following package was not found and will be installed

content/pt/docs/_index.md

-2
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,6 @@
22
title: Documentação
33
linkTitle: Docs
44
menu: { main: { weight: 10 } }
5-
htmltest:
6-
IgnoreDirs: {}
75
default_lang_commit: 2d88c10e1a14220a88a6e4859acb4047f49b6519
86
---
97

package.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,8 @@
7777
"fix": "npm run fix:all",
7878
"format": "npm run _check:format -- --write && npm run _check:format:ja+zh -- --write",
7979
"get:submodule": "npm run _get:${GET:-submodule} --",
80-
"log:check:links": "npm run check:links | tee tmp/build-log.txt",
81-
"log:test-and-fix": "npm run test-and-fix | tee tmp/build-log.txt",
80+
"log:check:links": "npm run check:links 2>&1 | tee tmp/build-log.txt",
81+
"log:test-and-fix": "npm run test-and-fix 2>&1 | tee tmp/build-log.txt",
8282
"make:public": "make public ls-public",
8383
"netlify-build:preview": "npm run seq -- build:preview diff:check",
8484
"netlify-build:production": "npm run seq -- build:production diff:check",

scripts/htmltest-config.pl

+3-3
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ sub extract_htmltest_config {
3838
return unless @htmltest_config;
3939

4040
if (@htmltest_config == 1) {
41-
warn "Warning: Failed to extract htmltest config from front matter in file '$file_path'.\n";
41+
warn "WARNING: Failed to extract htmltest config from front matter in file '$file_path'.\n";
4242
return;
4343
}
4444

@@ -51,7 +51,7 @@ sub extract_htmltest_config {
5151

5252
# TODO: Add support for `IgnoreURLs`.
5353

54-
warn "Warning: Unrecognized htmltest config from front matter in file '$file_path'.\n";
54+
warn "WARNING: Unrecognized htmltest config from front matter in file '$file_path'.\n";
5555
}
5656

5757
sub _extract_ignore_dirs {
@@ -67,7 +67,7 @@ sub _extract_ignore_dirs {
6767
} elsif ($line =~ /^IgnoreDirs:\s*\[\s*(.*?)\s*\]/i || $line =~ /^\s*-\s*(.*?)$/) {
6868
push @config, (split /\s*,\s*/, $1);
6969
} else {
70-
warn "Warning: Unrecognized htmltest IgnoreDirs config from front matter in file '$file_path': $line\n";
70+
warn "WARNING: Unrecognized htmltest IgnoreDirs config from front matter in file '$file_path': $line\n";
7171
}
7272
}
7373
return @config;

0 commit comments

Comments
 (0)