Skip to content

fix(tui): polish background tasks empty state copy#281

Open
ChuanWeis wants to merge 1 commit into
CortexLM:mainfrom
ChuanWeis:codex/cortex-background-tasks-copy
Open

fix(tui): polish background tasks empty state copy#281
ChuanWeis wants to merge 1 commit into
CortexLM:mainfrom
ChuanWeis:codex/cortex-background-tasks-copy

Conversation

@ChuanWeis
Copy link
Copy Markdown

Summary

  • Update the background tasks empty-state copy to the grammatical sentence No background tasks are running.
  • Move the empty-state copy into a small constant and add a focused regression test for the expected wording

Refs PlatformNetwork/bounty-challenge#53146.

Verification

  • git diff --check
  • Searched the TUI views for the old wording and confirmed it is no longer present

I could not run cargo fmt or cargo test locally because this Windows environment does not currently have cargo available on PATH.

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@ChuanWeis
Copy link
Copy Markdown
Author

Update: I installed the Rust toolchain locally and reran the targeted validation for this branch.

  • cargo test -p cortex-tui test_empty_tasks_message_is_grammatical passes.

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.

1 participant