Skip to content

Conversation

@iw-ezequiel
Copy link
Contributor

Some added context and clarification around the definitions of green/amber

Some added context and clarification around the definitions of green/amber
@iw-ezequiel iw-ezequiel self-assigned this Aug 9, 2021
@iw-ezequiel iw-ezequiel requested a review from a team August 9, 2021 12:01
Copy link
Contributor

@andyblundell andyblundell left a comment

Choose a reason for hiding this comment

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

To move this forward, how about we apply the general wording changes to the "usage" section in a brand new PR first, then we have another look at the table changes? I wouldn't mind including the this-is-what-red-really-means view of risks in the list too, which might be another good reason for formatting this a different way (not a table?) - and maybe we could do all of those table changes at the same time?

@iw-ezequiel
Copy link
Contributor Author

To move this forward, how about we apply the general wording changes to the "usage" section in a brand new PR first, then we have another look at the table changes? I wouldn't mind including the this-is-what-red-really-means view of risks in the list too, which might be another good reason for formatting this a different way (not a table?) - and maybe we could do all of those table changes at the same time?

Have created #207 for this, will leave this PR open in the meantime so as to cross-reference but will close once we've pulled all the content we need (because of the table it's kind of impossible to meaningfully cherry-pick anything!).

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 8, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@regularfry regularfry requested a review from a team as a code owner February 24, 2025 16:33
@sonarqubecloud
Copy link

@ghost
Copy link

ghost commented Feb 24, 2025

Dependabot Alert Summary

Total open alerts: 0
Alerts exceeding age threshold: 0

✅ All alerts are within acceptable age thresholds

@regularfry
Copy link
Contributor

Having resolved the merge conflicts, the majority of this PR has made its way in via other means. Closing.

@regularfry regularfry closed this Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants