Skip to content

Conversation

@Jefajers
Copy link
Member

@Jefajers Jefajers commented Jun 10, 2025

Overview/Summary

This PR adds specific DeploymentStacks deletion logic ensuring ActionOnUnmanage settings are honored during deletion of DeploymentStacks. Automated test cases are updated for coverage.

Also updating dependencies Az.Accounts to 5.1.0

This PR fixes/adds/changes/removes

  1. Changes AzOps.psd1
  2. Changes New-AzOpsDeployment.ps1
  3. Changes Remove-AzResourceRaw.ps1
  4. Changes Strings.psd1
  5. Changes Repository.Tests.ps1

Breaking Changes

N/A

Testing Evidence

Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant issues, for tracking and closure.
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.

@Jefajers Jefajers added this to the v2.8.1 milestone Jun 10, 2025
@Jefajers Jefajers requested a review from a team as a code owner June 10, 2025 18:53
@Jefajers Jefajers requested a review from a team as a code owner June 10, 2025 18:53
@Jefajers Jefajers added bug Something isn't working dependencies labels Jun 10, 2025
@Jefajers Jefajers moved this to In progress in AzOps Jun 10, 2025
@Jefajers Jefajers moved this from In progress to Needs review in AzOps Jun 11, 2025
@Jefajers Jefajers merged commit b67e79b into Azure:main Jun 18, 2025
4 checks passed
@Jefajers Jefajers deleted the stacki branch June 18, 2025 07:08
@Jefajers Jefajers moved this from Needs review to Pending release in AzOps Jun 18, 2025
@Jefajers Jefajers moved this from Pending release to Done in AzOps Jun 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working dependencies

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants