Skip to content

ai-did-you-mean-this CLI Extension#1535

Closed
christopher-o-toole wants to merge 9 commits intoAzure:masterfrom
christopher-o-toole:thoth-extension
Closed

ai-did-you-mean-this CLI Extension#1535
christopher-o-toole wants to merge 9 commits intoAzure:masterfrom
christopher-o-toole:thoth-extension

Conversation

@christopher-o-toole
Copy link
Contributor

@christopher-o-toole christopher-o-toole commented Apr 14, 2020 •
edited
Loading

No description provided.

@christopher-o-toole
Setup extension scaffolding, create dummy version command, add help/h…
8305ee4 
…istory/readme info.
@christopher-o-toole
Add v0 recommendation file. Handle general extension logic for when r…
3a33638 
…ecommendations aren't available and/or the user has an outdated CLI.
@christopher-o-toole
Fix empty iterable check, add guard against extension usage.
dfee69c
@christopher-o-toole
Disable linter warning for CLIError import, add more robust relative …
073bb20 
…pathing to recommendation file.
@christopher-o-toole
Add lookup table. Adjust minimum CLI version. Comment out parameter l…
794bb0c 
…ookup table logic for now.
@christopher-o-toole
Move "not able to help" message to function, accept list of parameter…
66c2d24 
…s, add Aladdin style parameter placeholders, remove lookup table references, resolved issue with generated recommendations.
@christopher-o-toole
Fix typo in comment.
82bd1f9
@christopher-o-toole
Remove comment.
f52ffbe
@christopher-o-toole
Clean up extension logic, adjust style based on flake8 and pylint.
47acc93
@azuresdkci
Copy link

azuresdkci commented Apr 14, 2020

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:

docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/christopher-o-toole/azure-cli-extensions.git@thoth-extension#subdirectory=src/$EXT&egg=$EXT"

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.

2 participants

@christopher-o-toole @azuresdkci