Skip to content

Comments

fix: Convert options from - to -- where it makes sense#329

Merged
alichtman merged 2 commits intoalichtman:mainfrom
BreakingPitt:fix_issue_#328
Aug 24, 2023
Merged

fix: Convert options from - to -- where it makes sense#329
alichtman merged 2 commits intoalichtman:mainfrom
BreakingPitt:fix_issue_#328

Conversation

@BreakingPitt
Copy link
Contributor

Fix: Convert options from - to -- where it makes sense

This pull request addresses an improvement in the command-line options for enhanced readability and alignment with conventions. The changes focus on converting short options (prefixed with -) to more descriptive long options (prefixed with --) where it makes sense.

Changes included in this pull request:

  • Converted short options to long options for improved clarity.
  • Ensured consistent usage of short and long options across the codebase.
  • Reviewed and updated option descriptions for accuracy.
  • Format code.

This enhancement simplifies the command-line interface and makes it more user-friendly.

Please review the changes and provide feedback.

Closes #328

@BreakingPitt
Copy link
Contributor Author

@alichtman It's there a way to know what's happening with the Codacy State Code Analysis stage?

@alichtman
Copy link
Owner

@alichtman It's there a way to know what's happening with the Codacy State Code Analysis stage?

Yeah, I'll check it out soon. Thanks for the PR! Looks good at first glance, but I need to look at it on my computer. Hard to actually review code on my phone 😂

@BreakingPitt
Copy link
Contributor Author

Sure, no need to review it your phone! xD xD xD

@alichtman alichtman merged commit af9c17b into alichtman:main Aug 24, 2023
@BreakingPitt BreakingPitt deleted the fix_issue_#328 branch August 24, 2023 21:32
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.

Convert options from - to -- where it makes sense

2 participants