-
Notifications
You must be signed in to change notification settings - Fork 129
Option appraisal pragmatic guide #392
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
Conversation
lhancock-scottlogic
left a comment
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.
Great read, I learned some stuff about architecture decision-making today!
As a more general suggestion, you could capitalise the important words on your main title, section titles, quick checklists, etc., i.e. Options appraisal: a pragmatic guide for architecture decisions ->
Options Appraisal: A pragmatic guide for Architecture Decisions
| - success is multi-dimensional (*cost* AND *speed* AND *risk* AND *operability*...) | ||
| - I can already see future-me asking "why did we choose this again?" | ||
|
|
||
| If it's a low-risk choice or you're still exploring, a quick spike, PoC, or desk-based research is often enough. |
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.
It may be worth writing out "proof of concept" just for the first time it's written in this post, I had to rack my brain a bit for the acronym's meaning, as it's not one I see every day 😁
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.
Fair enough, I have changed it to proof of concept :)
| | -------- | -------------------- | ------------- | ---------------- | ----- | ---------------------------------------------------------------------------- | | ||
| | Baseline | 5 → 15 | 1 → 5 | 4 → 8 | 28 | Known and operable, but weakest security posture | | ||
| | Option A | 3 → 9 | 3 → 15 | 4 → 8 | 32 | Balanced improvement with manageable operational cost | | ||
| | Option B | 1 → 3 | 5 → 25 | 1 → 2 | 30 | Pushes security to the maximum, but is the hardest option to live with day to day | |
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.
I'm not sure if it's intentional, but the column separator line between Operability and Total is thicker than the other lines
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.
It wasn’t intentional, no... You have eagle eyes! 👀😄
This looks like a quirk of markdown table rendering, but it actually works quite nicely here as it separates the criteria from the total. I’m inclined to leave it unless it’s distracting?
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.
That's very weird! I agree it looks good, I'm just wary that relying on "bug" behaviour can bite us in case it changes in the future - it would be nice to know why it's doing this so we can make sure it keeps doing it consistently
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.
I think is something to do with long then short column names. I would not know how to "fix" that to be honest.
Co-authored-by: lhancock-scottlogic <110816543+lhancock-scottlogic@users.noreply.github.com>
…points to improve the overall flow.
| - options-appraisal | ||
| - trade-offs | ||
| - architecture-decision-records | ||
| summary: A practical, lightweight way to compare options, surface trade-offs early, and leave a decision trail your future self will thank you for. |
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.
A thought here: it's not about something your future self will thank you for in our context, it's about something that the rest of the organisation will thank you for when you're no longer there. This would be better reworded to stress that it's for everyone's benefit, not just your own
| @@ -0,0 +1,196 @@ | |||
| --- | |||
| title: "Options appraisal: a pragmatic guide for architecture decisions" | |||
| date: 2025-12-19 00:00:00 Z | |||
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.
Update the date (here and in the filename) please
| | -------- | -------------------- | ------------- | ---------------- | ----- | ---------------------------------------------------------------------------- | | ||
| | Baseline | 5 → 15 | 1 → 5 | 4 → 8 | 28 | Known and operable, but weakest security posture | | ||
| | Option A | 3 → 9 | 3 → 15 | 4 → 8 | 32 | Balanced improvement with manageable operational cost | | ||
| | Option B | 1 → 3 | 5 → 25 | 1 → 2 | 30 | Pushes security to the maximum, but is the hardest option to live with day to day | |
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.
That's very weird! I agree it looks good, I'm just wary that relying on "bug" behaviour can bite us in case it changes in the future - it would be nice to know why it's doing this so we can make sure it keeps doing it consistently
Please add a direct link to your post here:
https://.github.io/blog/
Have you (please tick each box to show completion):
npm installfollowed bynpx mdspell "**/{FILE_NAME}.md" --en-gb -a -n -x -tif that's your thing)Posts are reviewed / approved by your Regional Tech Lead.