-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[clang][docs] Add clang-tutor to ExternalClangExamples. #107665
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@llvm/pr-subscribers-clang Author: None (c8ef) ChangesCloses #107584. Full diff: https://github.com/llvm/llvm-project/pull/107665.diff 1 Files Affected:
diff --git a/clang/docs/ExternalClangExamples.rst b/clang/docs/ExternalClangExamples.rst
index 8e986b83fd065f..301dbe9913329f 100644
--- a/clang/docs/ExternalClangExamples.rst
+++ b/clang/docs/ExternalClangExamples.rst
@@ -98,3 +98,6 @@ List of projects and tools
uses of reserved identifiers to ensuring that code adheres to lifecycle
protocols for certain LibreOffice-specific classes. They may serve as
examples for writing RecursiveASTVisitor-based plugins."
+
+`<https://github.com/banach-space/clang-tutor>`_
+ "A collection of out-of-tree Clang plugins for teaching and learning"
|
clang/docs/ExternalClangExamples.rst
Outdated
@@ -98,3 +98,6 @@ List of projects and tools | |||
uses of reserved identifiers to ensuring that code adheres to lifecycle | |||
protocols for certain LibreOffice-specific classes. They may serve as | |||
examples for writing RecursiveASTVisitor-based plugins." | |||
|
|||
`<https://github.com/banach-space/clang-tutor>`_ | |||
"A collection of out-of-tree Clang plugins for teaching and learning" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"A collection of out-of-tree Clang plugins for teaching and learning" | |
"A collection of out-of-tree Clang plugins for teaching and learning." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
Dear reviewers, if you the patch looks good to you, could you please assist me in merging it? |
Merged. Thank you for the addition, @c8ef! |
Closes #107584.