-
Notifications
You must be signed in to change notification settings - Fork 1
feat(provider): introduce environment management #322
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
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #322 +/- ##
==========================================
- Coverage 88.54% 88.17% -0.37%
==========================================
Files 181 194 +13
Lines 8505 9052 +547
==========================================
+ Hits 7531 7982 +451
- Misses 655 722 +67
- Partials 319 348 +29
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
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.
Pull Request Overview
This pull request adds support for managing Contentful environments as a Terraform resource. The implementation provides full CRUD operations for environment resources and includes corresponding test infrastructure.
- Adds
contentful_environmentresource with fields for space_id, environment_id, and name - Implements Create, Read, Update, Delete operations with proper version tracking
- Includes comprehensive test coverage with mocked Contentful API responses
Reviewed Changes
Copilot reviewed 43 out of 43 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| internal/provider/resource_environment.go | Core resource implementation with CRUD operations and identity schema |
| internal/provider/resource_environment_schema.go | Terraform schema definition for environment resource |
| internal/provider/environment_model.go | Data model structures for environment resource |
| internal/provider/environment_model_request.go | Request transformation from model to API fields |
| internal/provider/environment_model_response.go | Response transformation from API to model |
| internal/provider/resource_environment_test.go | Acceptance tests for resource operations and import |
| internal/provider/contentful_provider.go | Registration of new environment resource |
| internal/provider/testdata/TestAccEnvironmentResource/* | Terraform configuration files for tests |
| internal/provider/testdata/TestAccEnvironmentImport/* | Terraform configuration files for import tests |
| internal/contentful-management-go/testing/* | Mock server implementation for environment operations |
| internal/contentful-management-go/openapi/* | OpenAPI schema definitions for environment and environment alias |
| internal/contentful-management-go/oas_*_gen.go | Generated OpenAPI client code for environment operations |
| docs/resources/environment.md | Resource documentation with usage examples |
| examples/resources/contentful_environment/resource.tf | Example Terraform configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Pull Request Overview
Copilot reviewed 55 out of 55 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
0774349 to
8eb73c9
Compare
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.
Pull Request Overview
Copilot reviewed 57 out of 57 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
8eb73c9 to
b322764
Compare
b322764 to
26dd10e
Compare
26dd10e to
93542d8
Compare
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.
Pull Request Overview
Copilot reviewed 53 out of 53 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.