Skip to content

Remove short option "-t" for "--theme"#1576

Merged
shahzebsiddiqui merged 3 commits intodevelfrom
1571_remove_-t
Aug 2, 2023
Merged

Remove short option "-t" for "--theme"#1576
shahzebsiddiqui merged 3 commits intodevelfrom
1571_remove_-t

Conversation

@braeeast2001
Copy link
Collaborator

@braeeast2001 braeeast2001 commented Jul 25, 2023

To complete this issue the following will be done:

  • Update bash completion script and remove -t option wherever its used for --theme
  • Remove -t from documentation wherever its mentioned for --theme also check all .. command-output where its -t and replace with --theme
  • Update any docstring or comment throughout the code base

@braeeast2001 braeeast2001 self-assigned this Jul 25, 2023
@braeeast2001 braeeast2001 linked an issue Jul 25, 2023 that may be closed by this pull request
3 tasks
Removed "-t" where used in conjunction with "--theme"
@braeeast2001
Copy link
Collaborator Author

braeeast2001 commented Jul 25, 2023

Bash Script - Line 303
@shahzebsiddiqui

Should I remove the '-t' at the end of this line?

@braeeast2001 braeeast2001 changed the title Remove short option "-t" for "-theme" Remove short option "-t" for "--theme" Jul 31, 2023
Replace all -t for theme with --theme
@shahzebsiddiqui
Copy link
Member

@braeeast2001 can you confirm this task is complete. Is -t replaced to --theme

@braeeast2001
Copy link
Collaborator Author

@braeeast2001 can you confirm this task is complete. Is -t replaced to --theme

This task is complete.

Copy link
Member

@shahzebsiddiqui shahzebsiddiqui left a comment

Choose a reason for hiding this comment

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

lgtm

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.

Remove short option -t for --theme

2 participants