Skip to content

Remove default value for provider - #34

Merged
lionello merged 1 commit into
mainfrom
fix/remove-default-provider
Jan 21, 2026
Merged

Remove default value for provider#34
lionello merged 1 commit into
mainfrom
fix/remove-default-provider

Conversation

@raphaeltm

Copy link
Copy Markdown
Collaborator

The CLI will take care of determining the correct default provider

@raphaeltm
raphaeltm requested a review from a team as a code owner January 21, 2026 18:55
@raphaeltm
raphaeltm requested review from Copilot and removed request for a team January 21, 2026 18:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the hardcoded default value for the provider input parameter, allowing the Defang CLI to automatically determine the appropriate cloud provider when one is not explicitly specified.

Changes:

  • Updated the provider input description to clarify that the CLI will determine the provider if not specified
  • Changed the default value for provider from "defang" to an empty string
  • Modified the environment variable setting logic to conditionally set DEFANG_PROVIDER only when a provider is explicitly provided

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@lionello
lionello merged commit 827ecca into main Jan 21, 2026
10 checks passed
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.

3 participants