From e3ba25e45c2728fd6b1c91daacfcf7d0870ba6e9 Mon Sep 17 00:00:00 2001 From: Esther Kim <44627152+estherkim@users.noreply.github.com> Date: Tue, 3 Jan 2023 12:03:50 -0500 Subject: [PATCH] Skip validator-cpp tests for now (#38608) * test * skip * gh issue --- build-system/pr-check/validator-tests.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/build-system/pr-check/validator-tests.js b/build-system/pr-check/validator-tests.js index 90194261f617..35f1b0fcbacf 100644 --- a/build-system/pr-check/validator-tests.js +++ b/build-system/pr-check/validator-tests.js @@ -16,7 +16,8 @@ const jobName = 'validator-tests.js'; function pushBuildWorkflow() { timedExecOrDie('amp validator-webui'); timedExecOrDie('amp validator'); - timedExecOrDie('amp validator-cpp'); + // TODO(#38610): fix for bazel 6.0 or use older version + // timedExecOrDie('amp validator-cpp'); timedExecOrDie('amp validate-html-fixtures'); } @@ -53,9 +54,10 @@ function prBuildWorkflow() { timedExecOrDie('amp validator'); } - if (buildTargetsInclude(Targets.VALIDATOR)) { - timedExecOrDie('amp validator-cpp'); - } + // TODO(#38610): fix for bazel 6.0 or use older version + // if (buildTargetsInclude(Targets.VALIDATOR)) { + // timedExecOrDie('amp validator-cpp'); + // } if (buildTargetsInclude(Targets.HTML_FIXTURES)) { timedExecOrDie('amp validate-html-fixtures');