Skip to content
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.

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

[config] Add 'feature' subcommand #746

Merged
merged 9 commits into from
Nov 27, 2019
Merged

[config] Add 'feature' subcommand #746

merged 9 commits into from
Nov 27, 2019

Conversation

pra-moh
Copy link
Contributor

@pra-moh pra-moh commented Nov 20, 2019

- What I did
added command to configure feature status
- How I did it
By adding in main.py
- How to verify it
config feature_status name value
- Previous command output (if the output of a command-line utility has changed)

- New command output (if the output of a command-line utility has changed)

config/main.py Outdated Show resolved Hide resolved
config/main.py Outdated Show resolved Hide resolved
config/main.py Outdated Show resolved Hide resolved
changing argument name to state and adding click.choice
removed keyname
config/main.py Outdated Show resolved Hide resolved
config/main.py Outdated Show resolved Hide resolved
config/main.py Outdated Show resolved Hide resolved
@jleveque jleveque changed the title add config command feature_status [config] Add 'feature' subcommand Nov 21, 2019
pra-moh and others added 3 commits November 20, 2019 17:49
Co-Authored-By: Joe LeVeque <jleveque@users.noreply.github.com>
config/main.py Show resolved Hide resolved
config/main.py Outdated Show resolved Hide resolved
config/main.py Outdated Show resolved Hide resolved
config/main.py Outdated Show resolved Hide resolved
removing check for state and name empty
config/main.py Outdated Show resolved Hide resolved
config/main.py Outdated Show resolved Hide resolved
config/main.py Outdated Show resolved Hide resolved
@jleveque
Copy link
Contributor

Retest this please

@jleveque jleveque merged commit 0816948 into master Nov 27, 2019
@jleveque jleveque mentioned this pull request Dec 11, 2019
abdosi pushed a commit that referenced this pull request Feb 24, 2020
@qiluo-msft qiluo-msft deleted the addconfigfeature branch July 22, 2020 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants