Skip to content
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 info to CLI change summary #309

Merged
merged 1 commit into from
May 17, 2024
Merged

Add info to CLI change summary #309

merged 1 commit into from
May 17, 2024

Conversation

DavidS-ovm
Copy link
Contributor

@DavidS-ovm DavidS-ovm commented May 16, 2024

Also ensure that risks get printed at most 160 cells wide and add H2 style for risk titles

Fixes #306

2024-05-17_10MS+0200_631x412

@DavidS-ovm DavidS-ovm requested a review from dylanratcliffe May 16, 2024 14:38
Copy link
Member

@dylanratcliffe dylanratcliffe left a comment

Choose a reason for hiding this comment

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

Agreed on toning down the risk titles, let me have a bit of a play around with the styles

@DavidS-ovm DavidS-ovm force-pushed the wide-risks branch 3 times, most recently from 5940aba to efc94de Compare May 17, 2024 07:21
Also ensure that risks get printed at most 160 cells wide and add H2 style for risk titles

Fixes #306
@DavidS-ovm
Copy link
Contributor Author

I've used the color accent for the foreground instead of the background for the risk titles. That should do it for now. We can always revisit that closer to release.

@DavidS-ovm DavidS-ovm enabled auto-merge May 17, 2024 08:43
@DavidS-ovm DavidS-ovm merged commit 94aecaa into main May 17, 2024
5 checks passed
@DavidS-ovm DavidS-ovm deleted the wide-risks branch May 17, 2024 08:44
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.

Add info to CLI change summary
2 participants