Skip to content

Update README.md#5

Open
ajcw wants to merge 3 commits into
mainfrom
ajcw-patch-1
Open

Update README.md#5
ajcw wants to merge 3 commits into
mainfrom
ajcw-patch-1

Conversation

@ajcw
Copy link
Copy Markdown

@ajcw ajcw commented May 13, 2026

Emboldened text for easy scanning (part 1)

Emboldened text for easy scanning
@ajcw ajcw requested a review from robertpauldev May 13, 2026 15:25
Comment thread README.md Outdated
We ensure business critical **alerting** so that problems can be identified rapidly.

We step in to support the response to incidents involving our systems and ensure we document comprehensively to aid others.
We step in to **support the respons**e to incidents involving our systems and ensure we document comprehensively to aid others.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like the bold has broken the word response.

Fixed issue spotted by Bob and emboldened a few more words/phrases
@ajcw
Copy link
Copy Markdown
Author

ajcw commented May 14, 2026

Just testing Bob with a deliberate mistake. He passed the test so resubmitting.

robertpauldev
robertpauldev previously approved these changes May 14, 2026
Copy link
Copy Markdown
Member

@robertpauldev robertpauldev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😅 - LGTM @ajcw 🤙

Comment thread README.md Outdated
We put in place **logging and metrics** to monitor the health and performance of our systems and help with triaging and debugging of issues.

We ensure good coverage on logging and alerting so that problems can be identified rapidly.
We ensure business critical **alerting** so that problems can be identified rapidly.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm late to this party, and while I'd question the change of wording here (business-critical is an undefined moving target, aiming for "good coverage" would encompass that and other things), if we're going with that: I believe that business-critical should be hyphenated.

Happy to revert this back to "good coverage". Both phrases are subjective and ultimately I want this document to be objective, so we'll leave it as-is for now.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants