Skip to content

prevent ambiguous configuration for options#1392

Merged
pascalbreuninger merged 1 commit intoloft-sh:mainfrom
abdulari:abdulari/prevent-ambiguous-config
Dec 10, 2024
Merged

prevent ambiguous configuration for options#1392
pascalbreuninger merged 1 commit intoloft-sh:mainfrom
abdulari:abdulari/prevent-ambiguous-config

Conversation

@abdulari
Copy link
Copy Markdown
Contributor

prevent user from running ambiguous command such as:

vscode ➜ /workspaces/devpod (main) $ devpod provider options docker --provider docker2  --debug
10:05:21 info providerName=docker
10:05:21 info GlobalFlags.Provider=docker2
10:05:21 fatal ambiguous provider configuration detected
vscode ➜ /workspaces/devpod (main) $ 

@abdulari abdulari force-pushed the abdulari/prevent-ambiguous-config branch from 09b3c5f to 98f1663 Compare November 17, 2024 15:27
Copy link
Copy Markdown
Member

@pascalbreuninger pascalbreuninger left a comment

Choose a reason for hiding this comment

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

Fair point, LGTM! 👍

@pascalbreuninger pascalbreuninger merged commit bf111db into loft-sh:main Dec 10, 2024
@abdulari abdulari deleted the abdulari/prevent-ambiguous-config branch February 15, 2025 07:05
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.

2 participants