DeinAlptraum wrote: > Can you also add a test to ensure that nothing breaks?
What would you like to be tested? We already have tests asserting exact `CompletionString`s, see here: https://github.com/llvm/llvm-project/blob/077a796c0da6c32d5d43af9945b4c2728f43633f/clang/bindings/python/tests/cindex/test_code_completion.py#L51-L55 which I used to ensure that the format is correct (I kept adapting the converter function above until these tests passed) https://github.com/llvm/llvm-project/pull/160296 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
