-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add first version of C4 architecture diagrams #28
Conversation
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.
Welcome to our community 🤗 and thank you for your first contribution.
As a first time contributor please make sure to review our contribution guidelines ❤️
DescriptionCreates a `docs/architecture/ folder containing:
Resolves: https://github.com/orgs/MinBZK/projects/7/views/3?pane=issue&itemId=59900118 ChecklistPlease check all the boxes that apply to this pull request using "x":
|
First very minor remark, could you |
7256894
to
b68d3c8
Compare
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.
LGTM! Made some small remarks.
71c563e
to
1bc9b4d
Compare
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.
LGTM :)
1bc9b4d
to
10bfe55
Compare
Co-authored-by: Laurens Weijs <laurensweijs@gmail.com> Signed-off-by: GuusjeJuijn <96429015+GuusjeJuijn@users.noreply.github.com>
10bfe55
to
b753995
Compare
|
Description
Creates a `docs/architecture/ folder containing:
architecture.md
: a Markdown document containing C4-style diagrams for the Algorithm Management Toolkit. This document contains a system diagram, a container diagram and a component diagram, and a walkthrough and example text explaining the diagrams.Resolves: https://github.com/orgs/MinBZK/projects/7/views/3?pane=issue&itemId=59900118
Checklist
Please check all the boxes that apply to this pull request using "x":