equinix / terraform-provider-equinix

Terraform Equinix provider
https://deploy.equinix.com/labs/terraform-provider-equinix/
MIT License
47 stars 45 forks source link

refactor: move device resource and datasource to separate package #707

Closed ctreatma closed 1 week ago

ctreatma commented 3 months ago

This moves the device resource and data sources out of the equinix package and into an isolated directory & package to align with #106.

codecov-commenter commented 2 months ago

:warning: Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 80.95238% with 16 lines in your changes missing coverage. Please review.

Project coverage is 51.89%. Comparing base (f8c2ab9) to head (d698e10). Report is 9 commits behind head on main.

Files Patch % Lines
internal/resources/metal/device/resource.go 84.09% 6 Missing and 1 partial :warning:
internal/resources/metal/device/datasource.go 83.87% 4 Missing and 1 partial :warning:
internal/resources/metal/device/sweeper.go 0.00% 3 Missing :warning:
internal/resources/metal/device/helpers.go 0.00% 1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #707 +/- ## =========================================== + Coverage 34.44% 51.89% +17.45% =========================================== Files 151 152 +1 Lines 20710 20922 +212 =========================================== + Hits 7133 10857 +3724 + Misses 13415 9630 -3785 - Partials 162 435 +273 ```

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

github-actions[bot] commented 2 days ago

This PR is included in version 2.6.0 :tada: