-
Notifications
You must be signed in to change notification settings - Fork 8
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
Restructure introduction to specification #106
Conversation
Is this the right kind of thing? If so work to come out of this:
|
FYI this diagram is a refactor of @machintim 's original based on my understanding/what I think might be a bit clearer! |
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 subject to questions inline
@jemrobinson made edits, will leave open for a little bit to see if others have thoughts. Feel free to merge without me if not done today! |
As @harisood is away I will make the changes I've suggested above. However, please do say if you disagree because those are just my opinions after all. |
Co-authored-by: Matt Craddock <mcraddock@turing.ac.uk>
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.
If we want to restructure into three capabilities, shouldn't we also renumber the sections below?
Co-authored-by: Matt Craddock <monklefish@gmail.com>
Yes. Do you think it is in scope for this PR? Longer term, do we think that part will be on a separate page? |
I'm happy to do this in a follow-on PR. |
✅ Checklist
☑️ Maintainers' checklist
Refactor the introduction to the specification with our current version of the architecture, including visualisations
🌂 Related issues
🙋 Acknowledging contributors