search

pulumi / pulumi-terraform-bridge

A library allowing providers built with the Terraform Plugin SDK to be bridged into Pulumi.
Apache License 2.0
183 stars 41 forks source link

Refactor: Reduce pfutils in runtime usage #2076

Closed iwahbe closed 2 weeks ago

iwahbe commented 2 weeks ago

Built on top of #2075

This PR removes pfutil from github.com/pulumi/pulumi-terraform-bridge/pf.ShimProvider (introduced in #2075). There should be no runtime effects.

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 45.45455% with 30 lines in your changes missing coverage. Please review.

Project coverage is 61.20%. Comparing base (9cddce4) to head (ea3c33f).

Files Patch % Lines
pf/internal/pfutils/datasources.go 33.33% 4 Missing :warning:
pf/internal/schemashim/provider.go 20.00% 4 Missing :warning:
pf/tfbridge/provider_resources.go 0.00% 3 Missing :warning:
pf/internal/pfutils/common.go 60.00% 2 Missing :warning:
pf/internal/schemashim/datasource_map.go 0.00% 2 Missing :warning:
pf/tfbridge/provider_datasources.go 0.00% 2 Missing :warning:
pf/tfbridge/provider_invoke.go 0.00% 2 Missing :warning:
pf/tfbridge/resource_state.go 0.00% 2 Missing :warning:
pf/internal/pfutils/resources.go 87.50% 1 Missing :warning:
pf/internal/schemashim/datasource.go 0.00% 1 Missing :warning:
... and 7 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2076 +/- ## ========================================== - Coverage 62.01% 61.20% -0.81% ========================================== Files 338 339 +1 Lines 37524 45132 +7608 ========================================== + Hits 23270 27625 +4355 - Misses 12739 15987 +3248 - Partials 1515 1520 +5 ```

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