Merge stdlib #84
Merged
Merge stdlib #84
Conversation
This file contains hidden or 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
…e original appearance Signed-off-by: Michael Grolinger <[email protected]>
Improve the look of the legend table in LAYOUT_WITH_LEGEND() to better match the overall diagram style
Add diagram layout snippets
Improve legend
Change his to its for genderless lifeless object.
Fix grammar
fix: broken import C4_Container.puml link in README.md
fixed typos
Merged from plantuml-stdlib/C4-PlantUML/master
This was referenced Nov 22, 2020
stawirej
approved these changes
Nov 22, 2020
IOrlandoni
approved these changes
Nov 23, 2020
Member
|
@RicardoNiepel Could we get your final blessing here? |
Member
Author
|
Added one more commit to fix some broken links and to include the relationships from #86 in the readme file. |
|
Hello 👋 Curious about reasoning behind the decision to remove SystemDb and SystemDB_ext in favour of their Container versions. Thank you for this library 🙇 |
|
I also wonder about the rationale behind removing |
Member
|
@anorm Thank you for creating a separate issue for your question! @marcelstoer and @cabello shall we move this discussion there? ➡️ #113 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
14 participants
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.
Hi,
Apart from changes regarding plantUML compatibility this pull request includes most of the pull requests and issues.
I included @stawirej version of external containers/components. I will add intellij snippets later because the previous vscode snippet file changed a lot.
Queue containers/components have been added by @coryodaniel.
After this merge I can start tagging and checking out the release branch strategy mentioned in the previous discussions.
Edit: actually, there are many more changes: color fixes, arrowheads, deployment and dynamic diagrams, code snippets, bidirectional relationships, sprites, person icon.