E520 Add support for Bicep v0.36.177 by SimonWahlin · Pull Request #371 · PSBicep/PSBicep · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@SimonWahlin
Copy link
Member

Updated the Azure/Bicep dependency to version v0.36.177.

Copilot AI review requested due to automatic review settings July 9, 2025 20:00
Copy link
Copilot AI left a 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 microsoftGraph extension 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

@SimonWahlin SimonWahlin requested a review from PalmEmanuel July 9, 2025 20:05
@github-actions
Copy link
Contributor
github-actions bot commented Jul 9, 2025

Linux Test Results

213 tests  ±0   213 ✅ ±0   5s ⏱️ ±0s
 61 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit d4ffe64. ± Comparison against base commit a30e01c.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor
github-actions bot commented Jul 9, 2025

Win Test Results

213 tests  ±0   213 ✅ ±0   5s ⏱️ ±0s
 61 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit d4ffe64. ± Comparison against base commit a30e01c.

♻️ This comment has been updated with latest results.

@SimonWahlin SimonWahlin merged commit c28ba79 into main Jul 11, 2025
7 checks passed
@SimonWahlin SimonWahlin deleted the v0.36.177 branch July 11, 2025 05:59
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

0