Skip to content

Refactor project structure and add development tooling #281

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Refactor project structure and add development tooling #281

wants to merge 1 commit into from

Conversation

arthurbdiniz
Copy link

  • Add comprehensive DEVELOPMENT.md with project documentation
  • Create Makefile with build, test, lint and other development commands
  • Implement command structure using Cobra framework
  • Add man page generation support
  • Update .gitignore for new artifacts

@arthurbdiniz
Refactor project structure and add development tooling
1ea9c64 
- Add comprehensive DEVELOPMENT.md with project documentation
- Create Makefile with build, test, lint and other development commands
- Implement command structure using Cobra framework
- Add man page generation support
- Update .gitignore for new artifacts

Signed-off-by: Arthur Diniz <[email protected]>
@n-peugnet
Copy link
Contributor

I am not against these changes, but I would like to know what are the motivations behind rewriting the CLI with Cobra. What would you say are the main benefits?

@ottok
Copy link
Contributor

ottok commented Jun 19, 2025 •
edited
Loading

Same here. The commit message just summarizes what was done, but nothing about why this change is should be done.

@arthurbdiniz
Copy link
Author

By switching to Cobra, we gain some useful built-in features, such as shell auto-completion.

For example, users can enable Bash completion with:

source <(dh-make-golang completion bash)

Additionally, Cobra helps make the CLI more consistent and user-friendly by standardizing how flags and help messages are handled across commands.

This pull request also addresses inconsistent flag naming, some flags used dashes (-) while others used underscores (_). These are now unified.

cc @n-peugnet, @ottok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@arthurbdiniz @n-peugnet @ottok