-
Notifications
You must be signed in to change notification settings - Fork 37
Add support for Bicep v0.36.177 #371
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
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 updates the Bicep PowerShell module to depend on Bicep CLI v0.36.177, adds the AzResourceGraph module, and removes outdated private helper functions while updating calls and tests to align with these changes.
- Bump Azure/Bicep to v0.36.177 and add AzResourceGraph dependency
- Remove unused private cmdlets and switch to Search-AzResourceGraph with token support
- Rename TenantId → Tenant in Connect-Bicep and update associated tests
Reviewed Changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| Tests/Get-BicepConfig.Tests.ps1 | Adjust experimental feature flags and host lists in test fixture |
| Tests/Connect-Bicep.Tests.ps1 | Assert $Tenant instead of $TenantId and refine test names |
| Source/Public/Export-BicepResource.ps1 | Call Search-AzResourceGraph with -Token |
| Source/Public/Connect-Bicep.ps1 | Use script:TokenSplat['Tenant'] instead of TenantId |
| Source/Private/WriteErrorStream.ps1 | Remove unused WriteErrorStream helper |
| Source/Private/Search-AzureResourceGraph.ps1 | Remove unused custom ResourceGraph implementation |
| Source/Private/GetAzResourceGraphPage.ps1 | Remove unused paging helper |
| RequiredModules.psd1 | Bump AzAuth, add AzResourceGraph, update Azure/Bicep version |
| README.md | Add Features section and update command list |
| PSBicep.Core/Services/BicepRegistryService.cs | Refactor source archive packing and logging |
| PSBicep.Core/Configuration/bicepconfig.json | Update extensions and analyzer host lists |
| Docs/Help/Disconnect-Bicep.md | Add documentation for new Disconnect-Bicep cmdlet |
| CHANGELOG.md | Document removal of private functions and dependency changes |
| .github/actions/build/action.yml | Simplify SemVer regex for pre-release formatting |
Comments suppressed due to low confidence (2)
PSBicep.Core/Configuration/bicepconfig.json:34
- The
microsoftGraphextension entry was removed—this may unintentionally disable Microsoft Graph support. Please confirm whether dropping this extension is intended.
"kubernetes": "builtin:"
README.md:25
- The double hyphen (
--) here looks like a commented-out list item but may render incorrectly in Markdown. Consider removing or properly commenting this line if the command is intentionally hidden.
## Commands implemented
Updated the
Azure/Bicepdependency to versionv0.36.177.