Skip to content

Indent bullet points in error messages#1999

Merged
aknysh merged 2 commits intomainfrom
osterman/error-bullet-indent
Jan 20, 2026
Merged

Indent bullet points in error messages#1999
aknysh merged 2 commits intomainfrom
osterman/error-bullet-indent

Conversation

@osterman
Copy link
Copy Markdown
Member

what

  • Add 2-space indentation to bullet points in error messages for better visual hierarchy
  • Apply list indentation consistently in both TTY and ASCII rendering modes

why

  • Indented bullet lists are easier to scan and improve readability
  • Error messages with indented lists provide clearer visual structure to users

references

  • Related to error message styling improvements

Add list indentation to error message styles for better visual hierarchy.
Apply list indentation consistently across both TTY and ASCII rendering modes.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@osterman osterman requested a review from a team as a code owner January 19, 2026 23:04
@github-actions github-actions bot added the size/s Small size PR label Jan 19, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 19, 2026

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@github-actions github-actions bot added size/m Medium size PR and removed size/s Small size PR labels Jan 19, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jan 19, 2026

Warning

Rate limit exceeded

@osterman has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 10 minutes and 19 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 722a5c3 and 3a8fabc.

📒 Files selected for processing (17)
  • pkg/ui/markdown/renderer.go
  • pkg/ui/markdown/styles.go
  • tests/snapshots/TestCLICommands_atmos_about_non-existent.stderr.golden
  • tests/snapshots/TestCLICommands_atmos_atlantis.stderr.golden
  • tests/snapshots/TestCLICommands_atmos_atlantis_generate.stderr.golden
  • tests/snapshots/TestCLICommands_atmos_auth_invalid-command.stderr.golden
  • tests/snapshots/TestCLICommands_atmos_helmfile.stderr.golden
  • tests/snapshots/TestCLICommands_atmos_non-existent.stderr.golden
  • tests/snapshots/TestCLICommands_atmos_support.stdout.golden
  • tests/snapshots/TestCLICommands_atmos_terraform.stderr.golden
  • tests/snapshots/TestCLICommands_atmos_terraform_-help_passthrough.stdout.golden
  • tests/snapshots/TestCLICommands_atmos_terraform_non-existent.stderr.golden
  • tests/snapshots/TestCLICommands_atmos_workflow_invalid_from_step.stderr.golden
  • tests/snapshots/TestCLICommands_atmos_workflow_not_found.stderr.golden
  • tests/snapshots/TestCLICommands_atmos_workflow_pla_--file_workflow1.stderr.golden
  • tests/snapshots/TestCLICommands_describe_component_provenance_advanced.stdout.golden
  • tests/snapshots/TestCLICommands_describe_component_with_provenance_and_stack.stdout.golden
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch osterman/error-bullet-indent

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jan 19, 2026

Caution

Review failed

The head commit changed during the review from 91b0de7 to 3a8fabc.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch osterman/error-bullet-indent

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 19, 2026

Codecov Report

❌ Patch coverage is 84.61538% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.94%. Comparing base (722a5c3) to head (3a8fabc).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
pkg/ui/markdown/renderer.go 50.00% 2 Missing and 2 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1999      +/-   ##
==========================================
+ Coverage   74.90%   74.94%   +0.04%     
==========================================
  Files         777      777              
  Lines       71307    71333      +26     
==========================================
+ Hits        53413    53463      +50     
+ Misses      14412    14391      -21     
+ Partials     3482     3479       -3     
Flag Coverage Δ
unittests 74.94% <84.61%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
pkg/ui/markdown/styles.go 97.44% <100.00%> (+0.21%) ⬆️
pkg/ui/markdown/renderer.go 81.19% <50.00%> (-1.19%) ⬇️

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@osterman osterman added the no-release Do not create a new release (wait for additional code changes) label Jan 19, 2026
@aknysh aknysh merged commit cfd385e into main Jan 20, 2026
63 of 64 checks passed
@aknysh aknysh deleted the osterman/error-bullet-indent branch January 20, 2026 00:22
@github-actions
Copy link
Copy Markdown

These changes were released in v1.204.1-rc.3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-release Do not create a new release (wait for additional code changes) size/m Medium size PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants