llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang-tools-extra

Author: Congcong Cai (HerrCai0907)

<details>
<summary>Changes</summary>

Under previous test setup, the test result will be influenced by clang-tidy 
file in parent folder (between llvm-projects root and build folder). It is 
inconventient and leads some confusion.
This PR wants to ensure sandbox to avoid outside's clang-tidy influenece test 
result.


---
Full diff: https://github.com/llvm/llvm-project/pull/141410.diff


2 Files Affected:

- (added) clang-tools-extra/clangd/test/Inputs/path-mappings/server/.clang-tidy 
(+1) 
- (modified) clang-tools-extra/clangd/test/system-include-extractor.test (+3) 


``````````diff
diff --git 
a/clang-tools-extra/clangd/test/Inputs/path-mappings/server/.clang-tidy 
b/clang-tools-extra/clangd/test/Inputs/path-mappings/server/.clang-tidy
new file mode 100644
index 0000000000000..1f16f2a33b3ce
--- /dev/null
+++ b/clang-tools-extra/clangd/test/Inputs/path-mappings/server/.clang-tidy
@@ -0,0 +1 @@
+InheritParentConfig: false
\ No newline at end of file
diff --git a/clang-tools-extra/clangd/test/system-include-extractor.test 
b/clang-tools-extra/clangd/test/system-include-extractor.test
index 4ccc093671324..372b4a1a0fef5 100644
--- a/clang-tools-extra/clangd/test/system-include-extractor.test
+++ b/clang-tools-extra/clangd/test/system-include-extractor.test
@@ -32,6 +32,9 @@
 # RUN: echo 'printf "End of search list.\r\n" >&2' >> %t.dir/bin/my_driver.sh
 # RUN: chmod +x %t.dir/bin/my_driver.sh
 
+# Create fake clang-tidy config to ensure sandbox
+# RUN: echo 'InheritParentConfig: false' >> %t.dir/.clang-tidy
+
 # Create header files my/dir/a.h and my/dir2/b.h
 # RUN: mkdir -p %t.dir/my/dir
 # RUN: touch %t.dir/my/dir/a.h

``````````

</details>


https://github.com/llvm/llvm-project/pull/141410
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to