Skip to content

Conversation

@matt-FFFFFF
Copy link
Member

@matt-FFFFFF matt-FFFFFF commented Jun 6, 2025

@matt-FFFFFF matt-FFFFFF linked an issue Jun 6, 2025 that may be closed by this pull request
@matt-FFFFFF matt-FFFFFF requested review from Copilot and jtracey93 June 6, 2025 10:12
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 pull request fixes an issue with the policy sync script by addressing branch-related failures in the workflow and improving the script’s overall formatting and consistency.

  • Refactored conditional blocks to use consistent brace style.
  • Introduced an enforcement mode lookup dictionary to correctly map policy assignments.
  • Consolidated management group mappings and enhanced verbose logging for enforcement mode handling.

@matt-FFFFFF
Copy link
Member Author

/azp run unit

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@matt-FFFFFF
Copy link
Member Author

/azp run unit

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@matt-FFFFFF matt-FFFFFF merged commit c9d2c4d into main Jun 18, 2025
11 checks passed
@matt-FFFFFF matt-FFFFFF deleted the fix/policySync branch June 18, 2025 15:10
matt-FFFFFF added a commit that referenced this pull request Jun 18, 2025
Seemingly not all the required changes were included in #1286
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.

Bug Report - Faulty policy initiative Enforce Encryption CMK Still Broken, Requesting workaround

3 participants