Skip to content

Conversation

@netrounds-fredrik
Copy link
Contributor

This change makes the techn argument to Container and friends optional, similar to how it is optional in Rel.

I am not very familiar with this project or plantuml in general, so please review carefully.

This change makes the `techn` argument to `Container` and friends optional, similar to how it is optional in `Rel`.

I am not very familiar with this project or plantuml in general, so please review carefully.
@kirchsth kirchsth requested review from Potherca and kirchsth March 18, 2022 16:29
@kirchsth
Copy link
Member

Hi @netrounds-fredrik

thank you for the PR.
Optional $techn is a good finding/idea (In the meantime C4Model has container and components with optional technology too)

Can you please update the Component and documentation too?

Related to documentation:
I think it is enough if you delete the line
" *techn .. technology argument is required, ..."
and replace the remaining (2) *techn with ?techn

Thank you and best regards
Helmut

@kirchsth kirchsth removed the request for review from Potherca March 20, 2022 10:40
@kirchsth kirchsth added this to the v2.5.0 milestone Mar 20, 2022
@netrounds-fredrik
Copy link
Contributor Author

I have addressed the comments. Also made the changes to make the techn argument to Component optional to align the behavior.

@kirchsth kirchsth merged commit 19da5d5 into plantuml-stdlib:master Mar 21, 2022
@kirchsth
Copy link
Member

@netrounds-fredrik thank you for the PR
Helmut

@netrounds-fredrik netrounds-fredrik deleted the patch-1 branch March 22, 2022 09:21
@kirchsth kirchsth changed the title Make Container techn optional Make Container/Component techn optional Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants