-
Notifications
You must be signed in to change notification settings - Fork 454
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[Modules] Updated Policy Assignments to new dependency approach #1811
[Modules] Updated Policy Assignments to new dependency approach #1811
Conversation
Unit Test Results 1 files ± 0 1 suites ±0 1m 34s ⏱️ -8s Results for commit 2c6ce4d. ± Comparison against base commit 3b9c64e. This pull request removes 168 and adds 162 tests. Note that renamed tests count towards both.
This pull request removes 3 skipped tests and adds 9 skipped tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
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.
Thanks Alex. Some minor suggestions.
modules/Microsoft.Authorization/policyAssignments/.test/mg/deploy.test.bicep
Outdated
Show resolved
Hide resolved
modules/Microsoft.Authorization/policyAssignments/.test/rg.min/deploy.test.bicep
Outdated
Show resolved
Hide resolved
modules/Microsoft.Authorization/policyAssignments/.test/rg/dependencies.bicep
Outdated
Show resolved
Hide resolved
modules/Microsoft.Authorization/policyAssignments/.test/rg/deploy.test.bicep
Outdated
Show resolved
Hide resolved
modules/Microsoft.Authorization/policyAssignments/.test/rg/deploy.test.bicep
Outdated
Show resolved
Hide resolved
modules/Microsoft.Authorization/policyAssignments/.test/sub/deploy.test.bicep
Outdated
Show resolved
Hide resolved
@MrMCake please align the folder names to the proposed one from the role assignments module: |
Description
Pipeline references
Type of Change
Please delete options that are not relevant.