-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: add clang-uml dependency and test #120
Closed
Closed
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit
Hold shift + click to select a range
3e0e73b
feat: add clang-uml dependency and test
BarisTanyeri e6f262d
remove file exist assertion
BarisTanyeri e0cd062
try skip caching on certain run commands
BarisTanyeri 6ff074b
revert cache skip, add repo always before requirement
BarisTanyeri ce09a59
build & install clang-uml from source instead
BarisTanyeri File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
{ | ||
"ppa": "bkryza/clang-uml" | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,10 @@ | ||
{ | ||
"clang-15": "1:15.0.7-0ubuntu0.22.04.2", | ||
"clangd-15": "1:15.0.7-0ubuntu0.22.04.2", | ||
"clang-format-15": "1:15.0.7-0ubuntu0.22.04.2", | ||
"clang-tidy-15": "1:15.0.7-0ubuntu0.22.04.2", | ||
"clang-tools-15": "1:15.0.7-0ubuntu0.22.04.2", | ||
"lld-15": "1:15.0.7-0ubuntu0.22.04.2", | ||
"llvm-15": "1:15.0.7-0ubuntu0.22.04.2" | ||
"clang-15": "1:15.0.7-0ubuntu0.22.04.3", | ||
"clangd-15": "1:15.0.7-0ubuntu0.22.04.3", | ||
"clang-format-15": "1:15.0.7-0ubuntu0.22.04.3", | ||
"clang-tidy-15": "1:15.0.7-0ubuntu0.22.04.3", | ||
"clang-tools-15": "1:15.0.7-0ubuntu0.22.04.3", | ||
"lld-15": "1:15.0.7-0ubuntu0.22.04.3", | ||
"llvm-15": "1:15.0.7-0ubuntu0.22.04.3", | ||
"clang-uml": "0.3.8-0ubuntu1ppa1~jammy" | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
compilation_database_dir: build/clang | ||
output_directory: puml | ||
diagrams: | ||
test_class: | ||
type: class | ||
glob: | ||
- clang-tools/clang-uml-class.cpp | ||
comment_parser: clang | ||
using_namespace: | ||
- clanguml::t00002 | ||
include: | ||
namespaces: | ||
- clanguml::t00002 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
|
@@ -56,6 +56,13 @@ | |||||||||
"configurePreset": "clang", | ||||||||||
"targets": ["test-clang-iwyu"] | ||||||||||
}, | ||||||||||
{ | ||||||||||
"name": "clang-uml", | ||||||||||
"configurePreset": "clang", | ||||||||||
"targets": [ | ||||||||||
"test-clang-uml" | ||||||||||
] | ||||||||||
Comment on lines
+62
to
+64
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [MegaLinter] reported by reviewdog 🐶
Suggested change
|
||||||||||
}, | ||||||||||
{ | ||||||||||
"name": "coverage", | ||||||||||
"configurePreset": "coverage", | ||||||||||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,91 @@ | ||
#include <vector> | ||
|
||
namespace clanguml { | ||
namespace t00002 { | ||
|
||
/// \brief This is class A | ||
class A { | ||
public: | ||
/// Abstract foo_a | ||
virtual void foo_a() = 0; | ||
/// Abstract foo_c | ||
virtual void foo_c() = 0; | ||
}; | ||
|
||
/// \brief This is class B | ||
class B : public A { | ||
public: | ||
virtual void foo_a() override { } | ||
}; | ||
|
||
/// @brief This is class C - class C has a long comment | ||
/// | ||
/// Vivamus integer non suscipit taciti mus etiam at primis tempor sagittis sit, | ||
/// euismod libero facilisi aptent elementum felis blandit cursus gravida sociis | ||
/// erat ante, eleifend lectus nullam dapibus netus feugiat curae curabitur est | ||
/// ad. | ||
class C : public A { | ||
public: | ||
/// Do nothing unless override is provided | ||
virtual void foo_c() override { } | ||
}; | ||
|
||
/// This is class D | ||
/// which is a little like B | ||
/// and a little like C | ||
class D : public B, public C { | ||
public: | ||
/** | ||
* Forward foo_a | ||
*/ | ||
void foo_a() override | ||
{ | ||
for (auto a : as) | ||
a->foo_a(); | ||
} | ||
|
||
/** | ||
* Forward foo_c | ||
*/ | ||
void foo_c() override | ||
{ | ||
for (auto a : as) | ||
a->foo_c(); | ||
} | ||
|
||
private: | ||
/// All the A pointers | ||
std::vector<A *> as; | ||
}; | ||
|
||
class E : virtual public B, public virtual C { | ||
public: | ||
/// | ||
/// Forward foo_a | ||
/// | ||
void foo_a() override | ||
{ | ||
for (auto a : as) | ||
a->foo_a(); | ||
} | ||
|
||
/// | ||
/// Forward foo_c | ||
/// | ||
void foo_c() override | ||
{ | ||
for (auto a : as) | ||
a->foo_c(); | ||
} | ||
|
||
private: | ||
/// All the A pointers | ||
std::vector<A *> as; | ||
}; | ||
} // namespace t00002 | ||
} // namespace clanguml | ||
|
||
int main() | ||
{ | ||
return 0; | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
[MegaLinter] reported by reviewdog 🐶