Skip to content

Migrate to System.CommandLine for Bicep CLI#19312

Merged
anthony-c-martin merged 11 commits into
mainfrom
ant/cli
May 21, 2026
Merged

Migrate to System.CommandLine for Bicep CLI#19312
anthony-c-martin merged 11 commits into
mainfrom
ant/cli

Conversation

@anthony-c-martin
Copy link
Copy Markdown
Member

@anthony-c-martin anthony-c-martin commented Apr 1, 2026 •
edited by microsoft-github-policy-service Bot
Loading

Microsoft Reviewers: Open in CodeFlow

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2026 •
edited
Loading

Test this change out locally with the following install scripts (Action run 26256141072)

VSCode
  • Mac/Linux 
    bash <(curl -Ls https://aka.ms/bicep/nightly-vsix.sh) --run-id 26256141072
  • Windows 
    iex "& { $(irm https://aka.ms/bicep/nightly-vsix.ps1) } -RunId 26256141072"
Azure CLI
  • Mac/Linux 
    bash <(curl -Ls https://aka.ms/bicep/nightly-cli.sh) --run-id 26256141072
  • Windows 
    iex "& { $(irm https://aka.ms/bicep/nightly-cli.ps1) } -RunId 26256141072"

@anthony-c-martin anthony-c-martin force-pushed the ant/cli branch 2 times, most recently from 8e47ad9 to 2cbdfad Compare April 1, 2026 01:02
@anthony-c-martin
Merge remote-tracking branch 'origin/main' into ant/cli
c89d9c7 
# Conflicts:
#	src/Bicep.Cli/Arguments/SnapshotArguments.cs
#	src/Bicep.Cli/Bicep.Cli.csproj
@anthony-c-martin
Fix merge: add ManagementGroupId to snapshot command, fix help valida…
cc5b4c7 
…tor, restore package version
shenglol
shenglol reviewed May 21, 2026
View reviewed changes
Comment thread src/Bicep.Cli/Program.cs Outdated
@shenglol
Copy link
Copy Markdown
Contributor

shenglol commented May 21, 2026

These are out of the scope this PR and can be addressed later:

  • It looks like we don’t currently have e2e tests for the snapshot command.
  • It may be worth exploring the use of .NET Generic Host.

@anthony-c-martin anthony-c-martin mentioned this pull request May 21, 2026
Open
shenglol
shenglol approved these changes May 21, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@shenglol shenglol left a comment

Choose a reason for hiding this comment

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

:shipit:

@anthony-c-martin anthony-c-martin enabled auto-merge (squash) May 21, 2026 22:04
@anthony-c-martin anthony-c-martin merged commit 923dea3 into main May 21, 2026
41 checks passed
@anthony-c-martin anthony-c-martin deleted the ant/cli branch May 21, 2026 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shenglol shenglol shenglol approved these changes

Assignees

No one assigned

Labels

📘 Docs Not Needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@anthony-c-martin @shenglol