Skip to content

Add short option -t for --testpath#1599

Merged
shahzebsiddiqui merged 5 commits intodevelfrom
add_short_option_for_--testpath
Aug 25, 2023
Merged

Add short option -t for --testpath#1599
shahzebsiddiqui merged 5 commits intodevelfrom
add_short_option_for_--testpath

Conversation

@shahzebsiddiqui
Copy link
Member

No description provided.

braeeast2001 and others added 4 commits August 2, 2023 10:08
Add -t to line 1253 in __int__.py
Replace --testpath with -t in docs.py
Made replacements everywhere I could find `--testpath`
@shahzebsiddiqui shahzebsiddiqui merged commit 45089ef into devel Aug 25, 2023
@shahzebsiddiqui shahzebsiddiqui deleted the add_short_option_for_--testpath branch August 25, 2023 17:57
@jscook2345
Copy link

For documentation, I like to use long options so the user knows better what the option means, but I think this is fine to.

@shahzebsiddiqui
Copy link
Member Author

yeah i think either way works. This PR was addressing an issue where we had conflict with short option -t for --theme that was removed in #1576 so we reintroduced the -t for --testpath again. For consistency yeah we gotta have docs use long options and that would be a significant rewrite to docs. It may be a good thing. For now i used short option considering we were using -o for --output

@shahzebsiddiqui shahzebsiddiqui linked an issue Aug 30, 2023 that may be closed by this pull request
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

short option for 'buildtest inspect query'

3 participants