Skip to content

feat(cli): add show <command> subcommand with mock data#368

Merged
bobbyiliev merged 2 commits intoThe-DevOps-Daily:mainfrom
mt798jx:feat/show-command
Oct 3, 2025
Merged

feat(cli): add show <command> subcommand with mock data#368
bobbyiliev merged 2 commits intoThe-DevOps-Daily:mainfrom
mt798jx:feat/show-command

Conversation

@mt798jx
Copy link
Contributor

@mt798jx mt798jx commented Oct 1, 2025

  • Implemented show <command> subcommand to display description, usage, example, and notes
  • Added mock data for ls, grep, cat, mkdir
  • Improved output formatting with colors and sections
  • Added tests for valid and invalid commands in test_cli.py

Closes #331

Copy link
Collaborator

@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.

I think that this looks great! If you could just rebase from main we should be able to get it merged straight away!

@mt798jx
Copy link
Contributor Author

mt798jx commented Oct 1, 2025

it is done

@mt798jx
Copy link
Contributor Author

mt798jx commented Oct 1, 2025

u can approve it

@mt798jx mt798jx requested a review from bobbyiliev October 1, 2025 20:13
@mt798jx
Copy link
Contributor Author

mt798jx commented Oct 2, 2025

hey can u please close this PR and merge it? always they add new changes and I need always to rebase it

@bobbyiliev
Copy link
Collaborator

The lint seems to be failing: https://github.com/bobbyiliev/101-linux-commands/actions/runs/18189142243/job/51807286812?pr=368

Make sure to run it locally first and once it passes, we can merge it!

Also there are 14 commits here, make sure to rebase.

@mt798jx mt798jx force-pushed the feat/show-command branch from 5059623 to 878bead Compare October 2, 2025 17:49
feat(cli): add `show <command>` subcommand with mock data

- Implemented `show <command>` subcommand to display description, usage, example, and notes
- Added mock data for ls, grep, cat, mkdir
- Improved output formatting with colors and sections
- Added tests for valid and invalid commands in test_cli.py
@mt798jx mt798jx force-pushed the feat/show-command branch from 878bead to 10aa096 Compare October 2, 2025 17:53
@mt798jx
Copy link
Contributor Author

mt798jx commented Oct 2, 2025

Thanks for the feedback! 🙌
I’ve rebased the branch so there’s now just a single clean commit. I also ran the lint locally with: black --check cli/
and it passed successfully.
Additionally, I verified with pytest -v cli/test_cli.py and all tests passed ✅.

This should be good to go now 🚀.

@bobbyiliev
Copy link
Collaborator

Looks great! Really well done here @mt798jx! 👏

Thank you for the great contribution.

@bobbyiliev bobbyiliev merged commit 7c55eeb into The-DevOps-Daily:main Oct 3, 2025
4 checks passed
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 'show' command to display command details

2 participants