Skip to content

feat(add): preview well-known services with dry run#489

Open
jongio wants to merge 1 commit into
mainfrom
idea/add-dry-run-preview
Open

feat(add): preview well-known services with dry run#489
jongio wants to merge 1 commit into
mainfrom
idea/add-dry-run-preview

Conversation

@jongio

@jongio jongio commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Closes #486

Summary

  • add azd app add <service> --dry-run
  • print the generated azure.yaml service block without writing the file
  • include the preview in JSON output

Verification

  • go test .\src\cmd\app\commands -run "Test(BuildServicePreview|RunAddDryRunDoesNotModifyAzureYaml|AddServiceToYaml|ServiceExistsInYaml)"
  • mage test
  • mage build

@jongio jongio added the idea Feature idea from the idea pipeline label Jul 11, 2026
@jongio jongio self-assigned this Jul 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🚀 Website Preview

Your PR preview is ready!

📎 Preview URL: https://jongio.github.io/azd-app/pr/489/

This preview will be automatically cleaned up when the PR is closed.

github-actions Bot added a commit that referenced this pull request Jul 11, 2026
@codecov

codecov Bot commented Jul 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 63.88889% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.93%. Comparing base (68b3cae) to head (4a5aec9).

Files with missing lines Patch % Lines
cli/src/cmd/app/commands/add.go 63.88% 9 Missing and 4 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #489      +/-   ##
==========================================
+ Coverage   60.77%   60.93%   +0.15%     
==========================================
  Files         223      223              
  Lines       28527    28563      +36     
==========================================
+ Hits        17337    17404      +67     
+ Misses      10010     9968      -42     
- Partials     1180     1191      +11     
Flag Coverage Δ
unittests 60.93% <63.88%> (+0.15%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
cli/src/cmd/app/commands/add.go 55.93% <63.88%> (+20.82%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

Copy link
Copy Markdown
Contributor

🚀 Test This PR

A preview build (0.19.5-pr489) is ready for testing!

🌐 Website Preview

Live Preview: https://jongio.github.io/azd-app/pr/489/

One-Line Install (Recommended)

PowerShell (Windows):

iex "& { $(irm https://raw.githubusercontent.com/jongio/azd-app/main/cli/scripts/install-pr.ps1) } -PrNumber 489 -Version 0.19.5-pr489"

Bash (macOS/Linux):

curl -fsSL https://raw.githubusercontent.com/jongio/azd-app/main/cli/scripts/install-pr.sh | bash -s 489 0.19.5-pr489

Uninstall

When you're done testing:

PowerShell (Windows):

iex "& { $(irm https://raw.githubusercontent.com/jongio/azd-app/main/cli/scripts/uninstall-pr.ps1) } -PrNumber 489"

Bash (macOS/Linux):

curl -fsSL https://raw.githubusercontent.com/jongio/azd-app/main/cli/scripts/uninstall-pr.sh | bash -s 489

Build Info:

What to Test:
Please review the PR description and test the changes described there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

idea Feature idea from the idea pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a dry-run preview to azd app add

1 participant