fix: use test command for test flags

I don't think there's any reason we apply these two flags for `build`?
This commit is contained in:
Alex Eagle 2023-04-12 09:46:06 -07:00
parent e30284f2fa
commit adca277409
2 changed files with 4 additions and 4 deletions

View File

@ -3,7 +3,7 @@
# or split up into multiple test targets with sharding or manually. # or split up into multiple test targets with sharding or manually.
# Set this flag to exclude targets that have their timeout set to eternal (>15m) from running on CI. # Set this flag to exclude targets that have their timeout set to eternal (>15m) from running on CI.
# Docs: https://bazel.build/docs/user-manual#test-timeout-filters # Docs: https://bazel.build/docs/user-manual#test-timeout-filters
build --test_timeout_filters=-eternal test --test_timeout_filters=-eternal
# Set this flag to enable re-tries of failed tests on CI. # Set this flag to enable re-tries of failed tests on CI.
# When any test target fails, try one or more times. This applies regardless of whether the "flaky" # When any test target fails, try one or more times. This applies regardless of whether the "flaky"
@ -16,7 +16,7 @@ build --test_timeout_filters=-eternal
# is more likely to get fixed. # is more likely to get fixed.
# Note that when passing after the first attempt, Bazel will give a special "FLAKY" status. # Note that when passing after the first attempt, Bazel will give a special "FLAKY" status.
# Docs: https://bazel.build/docs/user-manual#flaky-test-attempts # Docs: https://bazel.build/docs/user-manual#flaky-test-attempts
build --flaky_test_attempts=2 test --flaky_test_attempts=2
# Announce all announces command options read from the bazelrc file(s) when starting up at the # Announce all announces command options read from the bazelrc file(s) when starting up at the
# beginning of each Bazel invocation. This is very useful on CI to be able to inspect what Bazel rc # beginning of each Bazel invocation. This is very useful on CI to be able to inspect what Bazel rc

View File

@ -3,7 +3,7 @@
# or split up into multiple test targets with sharding or manually. # or split up into multiple test targets with sharding or manually.
# Set this flag to exclude targets that have their timeout set to eternal (>15m) from running on CI. # Set this flag to exclude targets that have their timeout set to eternal (>15m) from running on CI.
# Docs: https://bazel.build/docs/user-manual#test-timeout-filters # Docs: https://bazel.build/docs/user-manual#test-timeout-filters
build --test_timeout_filters=-eternal test --test_timeout_filters=-eternal
# Set this flag to enable re-tries of failed tests on CI. # Set this flag to enable re-tries of failed tests on CI.
# When any test target fails, try one or more times. This applies regardless of whether the "flaky" # When any test target fails, try one or more times. This applies regardless of whether the "flaky"
@ -16,7 +16,7 @@ build --test_timeout_filters=-eternal
# is more likely to get fixed. # is more likely to get fixed.
# Note that when passing after the first attempt, Bazel will give a special "FLAKY" status. # Note that when passing after the first attempt, Bazel will give a special "FLAKY" status.
# Docs: https://bazel.build/docs/user-manual#flaky-test-attempts # Docs: https://bazel.build/docs/user-manual#flaky-test-attempts
build --flaky_test_attempts=2 test --flaky_test_attempts=2
# Announce all announces command options read from the bazelrc file(s) when starting up at the # Announce all announces command options read from the bazelrc file(s) when starting up at the
# beginning of each Bazel invocation. This is very useful on CI to be able to inspect what Bazel rc # beginning of each Bazel invocation. This is very useful on CI to be able to inspect what Bazel rc