Skip to content

Conversation

abhas20
Copy link
Contributor

@abhas20 abhas20 commented Oct 5, 2025

What type of PR is this? (check all applicable)

  • ♻️ Refactor
  • ✨ New Chapter
  • 🐛 Bug Fix/Typo
  • 👷 Optimization
  • 📝 Documentation Update
  • 🚩 Other

Description

  • Improved CLI UX by handling unknown commands.
  • Added a custom error running any unknown command will show a hint
  • Added test for the same

Before

Screenshot 2025-10-05 031824

After

image

I have added some screenshots for your reference please review it , and suggest any change or modification if needed.

Related Tickets & Documents

#332

Added to documentation?

  • 📜 readme
  • 🙅 no documentation needed

Copy link
Owner

@bobbyiliev bobbyiliev left a comment

Choose a reason for hiding this comment

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

Tests seem to be failing. If you could sort those out we could then merge the PR!

@abhas20
Copy link
Contributor Author

abhas20 commented Oct 7, 2025

Probably the issue was with the emoji added before 'Hint', leading to some color encoding errors, I have updated the test to handle it . Hope this will work.

@bobbyiliev bobbyiliev merged commit b3804bb into bobbyiliev:main Oct 8, 2025
2 checks passed
@abhas20 abhas20 deleted the feature branch October 8, 2025 13:25
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.

2 participants