From e125ead2b1549eff5bd20c3fb640c9480a9c9aeb Mon Sep 17 00:00:00 2001 From: Andrew Kelley Date: Tue, 15 Jun 2021 14:26:15 -0700 Subject: [PATCH] ci: remove confusing message now that --ast-check is added This extra message was intended to help contributors by clarifying what to do when they hit a `zig fmt` failure, but now AST errors are also emitted here and the message may actually introduce confusion. Remove it for now. --- ci/azure/linux_script | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ci/azure/linux_script b/ci/azure/linux_script index 462457a932..33c44c51e0 100755 --- a/ci/azure/linux_script +++ b/ci/azure/linux_script @@ -59,8 +59,9 @@ unset CXX make $JOBS install -# look for formatting errors -release/bin/zig fmt --check --ast-check .. || (echo "Please run 'zig fmt' to fix the non-conforming files listed above." && false) +# Look for formatting errors and AST errors. +# Formatting errors can be fixed by running `zig fmt` on the files printed here. +release/bin/zig fmt --check --ast-check .. # Here we rebuild zig but this time using the Zig binary we just now produced to # build zig1.o rather than relying on the one built with stage0. See