Skip to content

Fix bicep decompile-params if file is different name#17795

Merged
anthony-c-martin merged 2 commits into
Azure:mainfrom
Gijsreyn:fix/decompile-params-error
Aug 12, 2025
Merged

Fix bicep decompile-params if file is different name#17795
anthony-c-martin merged 2 commits into
Azure:mainfrom
Gijsreyn:fix/decompile-params-error

Conversation

@Gijsreyn
Copy link
Copy Markdown
Contributor

@Gijsreyn Gijsreyn commented Aug 12, 2025

Description

Related issue #14092

Example Usage

Whenever a user attempts to decompile a parameter file with a different name, it still throws that the file already exist and --force should be used. Whenever --force is applied, the file is generated with the different name.

image

This change removes the additional check. Validated with latest build locally:

image

Checklist

Microsoft Reviewers: Open in CodeFlow

Copy link
Copy Markdown
Member

@anthony-c-martin anthony-c-martin left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this!

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 9.37%. Comparing base (108d816) to head (2d06456).
⚠️ Report is 2944 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (108d816) and HEAD (2d06456). Click for more details.

HEAD has 4 uploads less than BASE
Flag BASE (108d816) HEAD (2d06456)
dotnet 4 0
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main   #17795       +/-   ##
===========================================
- Coverage   94.28%    9.37%   -84.91%     
===========================================
  Files        1113        7     -1106     
  Lines      100791      288   -100503     
  Branches     8734      121     -8613     
===========================================
- Hits        95028       27    -95001     
+ Misses       4595      261     -4334     
+ Partials     1168        0     -1168     
Flag Coverage Δ
dotnet ?
typescript 9.37% <ø> (+1.66%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1103 files with indirect coverage changes

🚀 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.

@anthony-c-martin anthony-c-martin merged commit 3c8ec85 into Azure:main Aug 12, 2025
40 of 41 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

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

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

@Gijsreyn Gijsreyn deleted the fix/decompile-params-error branch August 15, 2025 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants