microsoft / terraform-provider-power-platform

Power Platform Terraform Provider
https://registry.terraform.io/providers/microsoft/power-platform/latest/docs
MIT License
35 stars 13 forks source link

Adding powerplatform_environment_group_rule_set #501

Closed mawasile closed 3 weeks ago

mawasile commented 4 weeks ago

This pull request introduces a new resource for managing Power Platform environment group rule sets, along with several supporting changes across the codebase. The most important changes include the addition of the new resource, updates to various files to support this new resource, and improvements to existing code for better functionality and maintainability.

New Resource Addition:

Supporting Changes for New Resource:

Code Improvements:

Minor Fixes:

These changes collectively enhance the functionality and maintainability of the codebase by introducing new features and improving existing code.

codecov-commenter commented 4 weeks ago

Codecov Report

Attention: Patch coverage is 72.91221% with 253 lines in your changes missing coverage. Please review.

Project coverage is 73.78%. Comparing base (a2344f2) to head (20d4ea4). Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
...nternal/services/environment_group_rule_set/dto.go 78.70% 73 Missing and 32 partials :warning:
...up_rule_set/resource_environment_group_rule_set.go 72.46% 75 Missing and 12 partials :warning:
...t_group_rule_set/api_environment_group_rule_set.go 52.57% 38 Missing and 8 partials :warning:
internal/helpers/uri.go 0.00% 12 Missing :warning:
internal/services/connection/api_connection.go 77.77% 2 Missing :warning:
internal/api/client.go 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #501 +/- ## ========================================== - Coverage 73.90% 73.78% -0.13% ========================================== Files 98 102 +4 Lines 13521 14434 +913 ========================================== + Hits 9993 10650 +657 - Misses 2921 3123 +202 - Partials 607 661 +54 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.