Skip to content

Conversation

@adeeshperera
Copy link
Contributor

@adeeshperera adeeshperera commented Oct 7, 2025

  • Removed emojis from section headers, bullet points, and status messages
  • Updated display.go and createMsg.go to use plain text instead of emojis
image

Summary by CodeRabbit

  • New Features

    • Added per-category file lists in Changes Summary: shows first 5 staged files and first 3 unstaged/untracked, with “… and N more” when applicable.
    • Maintains totals and a compact commit message section; stats table remains when additions/deletions are present.
  • Style

    • Removed emojis/icons from titles and messages for a cleaner, plain-text look.
    • Simplified success and clipboard messages to straightforward text for consistency.

adeeshperera and others added 2 commits October 7, 2025 09:09
…pearance

- Removed emojis from section headers, bullet points, and status messages
- Updated display.go and createMsg.go to use plain text instead of emojis
fix DFanso#69 : remove emojis from console output for professional appearance
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 7, 2025

Walkthrough

This PR removes emoji/icons from CLI and display messages and revises display rendering to include per-category file lists with truncation for staged, unstaged, and untracked files. Section headers are updated to plain text. No exported APIs or control flow are changed.

Changes

Cohort / File(s) Summary
CLI message text updates
cmd/cli/createMsg.go
Replaced emoji-prefixed success and clipboard messages with plain-text equivalents. No logic changes.
Display headers and file list rendering
internal/display/display.go
Removed emoji from headers/labels. Added per-category file lists with truncation (staged: first 5; unstaged/untracked: first 3) plus “… and N more”. Retained existing box/table rendering; no exported signatures changed.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User
  participant CLI as CLI (createMsg)
  participant Display as Display Renderer
  participant TTY as Terminal

  User->>CLI: Generate commit message
  CLI->>Display: ShowFileStatistics(staged, unstaged, untracked)
  rect rgba(230, 240, 255, 0.6)
    note right of Display: Render plain-text headers
    Display->>Display: List staged (max 5) + “… and N more”
    Display->>Display: List unstaged (max 3) + “… and N more”
    Display->>Display: List untracked (max 3) + “… and N more”
  end
  Display->>TTY: Print summary + lists

  CLI->>Display: ShowCommitMessage(message)
  Display->>TTY: Print compact header + boxed message

  CLI->>Display: ShowChangesPreview(stats?)
  alt Stats available
    Display->>TTY: Render 2-column info table
  else No stats
    Display->>TTY: Render no-stats message
  end
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

hacktoberfest, hacktoberfest-accepted

Poem

I hop through diffs, no icons in my trail,
Plain words now sparkle—concise, never stale.
Staged, unstaged, untracked in tidy rows,
“...and N more” whispers what overflow shows.
Commit box gleams, previews neat—
A rabbit approves, with tapping feet. 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 75.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title clearly summarizes the main change by indicating the removal of emojis from console output to address issue #69 and enhance professional appearance, directly reflecting the modifications in the pull request.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9e41121 and d9cee41.

📒 Files selected for processing (2)
  • cmd/cli/createMsg.go (2 hunks)
  • internal/display/display.go (5 hunks)
🔇 Additional comments (2)
cmd/cli/createMsg.go (1)

154-154: LGTM! Emoji removals align with PR objective.

The removal of emojis from the success and warning messages achieves a more professional console output while maintaining message clarity. All three changes are straightforward string updates with no logic modifications.

Also applies to: 164-164, 166-166

internal/display/display.go (1)

21-21: LGTM! Consistent emoji removal across all display functions.

All section headers and status labels have been successfully updated to remove emoji prefixes:

  • "Changes Summary" (line 21)
  • File status labels for staged/unstaged/untracked (lines 29, 52, 75)
  • "Generated Commit Message" (line 100)
  • "Changes Preview" (line 119)

The changes maintain a professional appearance while preserving all existing functionality, color coding, and file listing behavior.

Also applies to: 29-29, 52-52, 75-75, 100-100, 119-119


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

@DFanso DFanso added bug Something isn't working enhancement New feature or request hacktoberfest Eligible for Hacktoberfest hacktoberfest-accepted Approved Hacktoberfest contribution go Pull requests that update go code labels Oct 7, 2025
@DFanso DFanso linked an issue Oct 7, 2025 that may be closed by this pull request
@DFanso DFanso self-requested a review October 7, 2025 03:57
Copy link
Owner

@DFanso DFanso left a comment

Choose a reason for hiding this comment

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

Approved 👾

@DFanso DFanso merged commit 86f674b into DFanso:main Oct 7, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request go Pull requests that update go code hacktoberfest Eligible for Hacktoberfest hacktoberfest-accepted Approved Hacktoberfest contribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Remove emojis from console output for a more professional appearance

2 participants