dikhan / terraform-provider-openapi

OpenAPI Terraform Provider that configures itself at runtime with the resources exposed by the service provider (defined in a swagger file)
Apache License 2.0
275 stars 48 forks source link

[FeatureRequest: Issue #224] Add telemetry to data sources #225

Closed dikhan closed 4 years ago

dikhan commented 4 years ago

What problem does this Pull Request solve?

Please link to the issue number here: #224

If you don't have an issue created, please create the issue first describing the problem and then open the PR

Type of change

What type of change does your code introduce to the provider? Please put an x (w/o heading/trailing white spaces) in the boxes that apply:

Checklist

Please put an x (w/o heading/trailing white spaces) in the boxes that apply:

Checklist for Admins

codecov[bot] commented 4 years ago

Codecov Report

Merging #225 into master will increase coverage by 0.07%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #225      +/-   ##
==========================================
+ Coverage   90.55%   90.63%   +0.07%     
==========================================
  Files          56       56              
  Lines        3283     3310      +27     
==========================================
+ Hits         2973     3000      +27     
  Misses        205      205              
  Partials      105      105              
Impacted Files Coverage Δ
openapi/data_source_factory.go 98.16% <100.00%> (+0.06%) :arrow_up:
openapi/data_source_instance_factory.go 100.00% <100.00%> (ø)
openapi/plugin_config_telemetry_handler.go 100.00% <100.00%> (ø)
openapi/resource_factory.go 90.79% <100.00%> (+0.24%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 6f2c3a9...b52901e. Read the comment docs.