From 29c0c28a2c13f898c8245250aab8f5f336250a91 Mon Sep 17 00:00:00 2001 From: Sam Xie Date: Thu, 5 Sep 2024 12:45:47 -0700 Subject: [PATCH] Mention `test-concurrent-safe` CI job in CONTRIBUTING doc (#5781) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Related #5759 --------- Co-authored-by: Robert Pająk --- CONTRIBUTING.md | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index b7402576f98..91580725350 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -578,7 +578,10 @@ See also: The tests should never leak goroutines. Use the term `ConcurrentSafe` in the test name when it aims to verify the -absence of race conditions. +absence of race conditions. The top-level tests with this term will be run +many times in the `test-concurrent-safe` CI job to increase the chance of +catching concurrency issues. This does not apply to subtests when this term +is not in their root name. ### Internal packages