[clang-tools-extra] [llvm] [clang-doc] Add basic e2e test (PR #93928)

2024-06-06 Thread Paul Kirth via cfe-commits
ilovepi wrote: Also, the current PR only has tests for the HTML output. Please file separate bugs for the other formats, and be sure to follow up with more tests for those formats. https://github.com/llvm/llvm-project/pull/93928 ___ cfe-commits maili

[clang-tools-extra] [llvm] [clang-doc] Add basic e2e test (PR #93928)

2024-06-06 Thread Paul Kirth via cfe-commits
@@ -186,19 +196,36 @@ Example usage for a project using a compile commands database: std::string ClangDocPath = GetExecutablePath(argv[0], MainAddr); llvm::SmallString<128> NativeClangDocPath; llvm::sys::path::native(ClangDocPath, NativeClangDocPath); -llvm::Sm

[clang-tools-extra] [llvm] [clang-doc] Add basic e2e test (PR #93928)

2024-06-06 Thread Paul Kirth via cfe-commits
@@ -186,19 +196,36 @@ Example usage for a project using a compile commands database: std::string ClangDocPath = GetExecutablePath(argv[0], MainAddr); llvm::SmallString<128> NativeClangDocPath; llvm::sys::path::native(ClangDocPath, NativeClangDocPath); -llvm::Sm

[clang-tools-extra] [llvm] [clang-doc] Add basic e2e test (PR #93928)

2024-06-06 Thread Paul Kirth via cfe-commits
@@ -131,6 +135,12 @@ std::string GetExecutablePath(const char *Argv0, void *MainAddr) { return llvm::sys::fs::getMainExecutable(Argv0, MainAddr); } +inline bool ends_with(std::string const & value, std::string const & ending) +{ +if (ending.size() > value.size()) return

[clang-tools-extra] [llvm] [clang-doc] Add basic e2e test (PR #93928)

2024-06-06 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff 98714866830f505d7bb87de6b92a28f280a34b9b 5a647f015051a572a36af6418fadb424db470cec --

[clang-tools-extra] [llvm] [clang-doc] Add basic e2e test (PR #93928)

2024-06-06 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/93928 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [llvm] [clang-doc] Add basic e2e test (PR #93928)

2024-06-06 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi commented: All of the changes related to allowing a different path belong in their own PR, or if they're invasive, perhaps multiple PRs. https://github.com/llvm/llvm-project/pull/93928 ___ cfe-commits mailing list cfe-commit

[clang-tools-extra] [llvm] [clang-doc] Add basic e2e test (PR #93928)

2024-06-06 Thread Paul Kirth via cfe-commits
@@ -69,8 +70,15 @@ clang_tidy_headers = os.path.join( config.test_source_root, "clang-tidy", "checkers", "Inputs", "Headers" ) + config.substitutions.append(("%clang_tidy_headers", clang_tidy_headers)) +clang_doc_install = os.path.join( +os.path.dirname(shutil.which(

[clang-tools-extra] [llvm] [clang-doc] Add basic e2e test (PR #93928)

2024-06-06 Thread via cfe-commits
@@ -69,8 +70,15 @@ clang_tidy_headers = os.path.join( config.test_source_root, "clang-tidy", "checkers", "Inputs", "Headers" ) + config.substitutions.append(("%clang_tidy_headers", clang_tidy_headers)) +clang_doc_install = os.path.join( +os.path.dirname(shutil.which(

[clang-tools-extra] [llvm] [clang-doc] Add basic e2e test (PR #93928)

2024-06-06 Thread Paul Kirth via cfe-commits
@@ -69,8 +70,15 @@ clang_tidy_headers = os.path.join( config.test_source_root, "clang-tidy", "checkers", "Inputs", "Headers" ) + config.substitutions.append(("%clang_tidy_headers", clang_tidy_headers)) +clang_doc_install = os.path.join( +os.path.dirname(shutil.which(

[clang-tools-extra] [llvm] [clang-doc] Add basic e2e test (PR #93928)

2024-06-06 Thread via cfe-commits
https://github.com/PeterChou1 edited https://github.com/llvm/llvm-project/pull/93928 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [llvm] [clang-doc] Add basic e2e test (PR #93928)

2024-06-06 Thread via cfe-commits
https://github.com/PeterChou1 edited https://github.com/llvm/llvm-project/pull/93928 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits