-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat cloud cost credentials #218 #252
Conversation
…into feat_cloud_cost_credentials-#218
…into feat_cloud_cost_credentials-#218
…/env0/terraform-provider-env0 into feat_cloud_cost_credentials-#218
…/env0/terraform-provider-env0 into feat_cloud_cost_credentials-#218
…/env0/terraform-provider-env0 into feat_cloud_cost_credentials-#218
…into feat_cloud_cost_credentials-#218
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.
I'm sorry @samuel-br but this PR is wayyy to big for someone to review...
please split this pr
one way to split it is by
- pr for api client
- pr for resources
and please try to make the code a bit more generic, maybe the cost model can contain everything with omit empty so we can use the same common code for all the resource but with different schema
LMK if you need help
Closing this one as we you open the splits |
Issue & Steps to Reproduce / Feature Request
resolves #218
Solution
Add the relevant resources and data