Skip to content

Conversation

@keerthivasan2002
Copy link

@keerthivasan2002 keerthivasan2002 commented Sep 29, 2025

Fix CLI: add generate-tests and generate-examples aliases

Issue

Users currently have to type pdd test or pdd example explicitly.
It would be more natural to allow pdd generate-tests ... and pdd generate-examples ... as shorthand.

Solution

Added convenience aliases to support the following:

  • pdd generate-tests ... → maps to pdd test ...
  • pdd generate-examples ... → maps to pdd example ...

This improves ergonomics without breaking existing commands.

Key Files Changed

Core Code:

  • pdd/cli.py — added alias registration with cli.add_command.

Documentation:

  • README.md — updated Command Overview section to list the new aliases.

@gltanaka gltanaka requested a review from Copilot September 29, 2025 07:37
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds convenience aliases for the test and example commands to improve CLI ergonomics. Users can now use more descriptive command names like pdd generate-tests and pdd generate-examples instead of the shorter pdd test and pdd example.

Key changes:

  • Added alias registration in CLI module to map new command names to existing functionality
  • Updated documentation to reflect the new command aliases

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
pdd/cli.py Added alias registration for generate-tests and generate-examples commands
README.md Updated command overview section to document the new aliases

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@rudi-cilibrasi
Copy link
Contributor

Why did you add hyphens? Wouldn't it be easier just sticking with spaces to separate commands?

@keerthivasan2002
Copy link
Author

I went with hyphens since many CLI commands use them for multi-word subcommands, but I’m open to switching to spaces if that improves usability.

@gltanaka gltanaka requested a review from Copilot September 30, 2025 18:39
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@keerthivasan2002
Copy link
Author

Fixes #56

@gltanaka
Copy link
Contributor

@keerthivasan2002 has this been tested?

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.

3 participants