[CI] Ensure that htmltest-config warnings fail GH check (open-telemet…

…ry#5612)
ymotongpoo · Jan 2, 2025 · 56cf671 · 56cf671
1 parent 1a74825
commit 56cf671
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 7 deletions.
diff --git a/.warnings-skip-list.txt b/.warnings-skip-list.txt
@@ -1 +1,2 @@
 _filename-error
+The following package was not found and will be installed
diff --git a/content/pt/docs/_index.md b/content/pt/docs/_index.md
@@ -2,8 +2,6 @@
 title: Documentação
 linkTitle: Docs
 menu: { main: { weight: 10 } }
-htmltest:
-  IgnoreDirs: {}
 default_lang_commit: 2d88c10e1a14220a88a6e4859acb4047f49b6519
 ---
 

diff --git a/package.json b/package.json
@@ -77,8 +77,8 @@
     "fix": "npm run fix:all",
     "format": "npm run _check:format -- --write && npm run _check:format:ja+zh -- --write",
     "get:submodule": "npm run _get:${GET:-submodule} --",
-    "log:check:links": "npm run check:links | tee tmp/build-log.txt",
-    "log:test-and-fix": "npm run test-and-fix | tee tmp/build-log.txt",
+    "log:check:links": "npm run check:links 2>&1 | tee tmp/build-log.txt",
+    "log:test-and-fix": "npm run test-and-fix 2>&1 | tee tmp/build-log.txt",
     "make:public": "make public ls-public",
     "netlify-build:preview": "npm run seq -- build:preview diff:check",
     "netlify-build:production": "npm run seq -- build:production diff:check",

diff --git a/scripts/htmltest-config.pl b/scripts/htmltest-config.pl
@@ -38,7 +38,7 @@ sub extract_htmltest_config {
     return unless @htmltest_config;
 
     if (@htmltest_config == 1) {
-        warn "Warning: Failed to extract htmltest config from front matter in file '$file_path'.\n";
+        warn "WARNING: Failed to extract htmltest config from front matter in file '$file_path'.\n";
         return;
     }
 
@@ -51,7 +51,7 @@ sub extract_htmltest_config {
 
     # TODO: Add support for `IgnoreURLs`.
 
-    warn "Warning: Unrecognized htmltest config from front matter in file '$file_path'.\n";
+    warn "WARNING: Unrecognized htmltest config from front matter in file '$file_path'.\n";
 }
 
 sub _extract_ignore_dirs {
@@ -67,7 +67,7 @@ sub _extract_ignore_dirs {
       } elsif ($line =~ /^IgnoreDirs:\s*\[\s*(.*?)\s*\]/i || $line =~ /^\s*-\s*(.*?)$/) {
         push @config, (split /\s*,\s*/, $1);
       } else {
-        warn "Warning: Unrecognized htmltest IgnoreDirs config from front matter in file '$file_path': $line\n";
+        warn "WARNING: Unrecognized htmltest IgnoreDirs config from front matter in file '$file_path': $line\n";
       }
     }
     return @config;
Original file line number	Diff line number	Diff line change
		@@ -1 +1,2 @@
		_filename-error
		The following package was not found and will be installed