Replies: 2 comments
-
Thanks for the feedback - I've made some tweaks. |
Beta Was this translation helpful? Give feedback.
0 replies
-
Nice. Should hopefully help to reduce confusion in the future. Thanks! |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hi Simon,
thanks again for all your effort put into the C4 Model & Structurizr.
I was just now reading about the implied relationships documentation here and followed the link to here as well. Maybe it's just me, but I found a bit it confusing.
In the first link you write
and that this behaviour can be disabled via
!impliedRelationships
set tofalse
. But when following the second link, you stateBy default, the Structurizr for Java library will not create implied relationships.
.I guess since dsl handles it different than the java library, maybe it would be good to make it more explicit i.e.
By default, the Structurizr for Java library will not create implied relationships. In contrary to the DSL which uses CreateImpliedRelationshipsUnlessAnyRelationshipExistsStrategy as default.
Just my 2 cents.
Additional to this, here you are talking about
CreateImpliedRelationshipsUnlessSameRelationshipExistsStrategy
but in the example codeCreateImpliedRelationshipsUnlessAnyRelationshipExistsStrategy
is used. Is this intended?Thanks and have a nice day
Beta Was this translation helpful? Give feedback.
All reactions