Skip to content
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

refactor: setup muxing to allow migrating to plugin-framework #531

Merged
merged 12 commits into from
May 7, 2024

Conversation

kangasta
Copy link
Contributor

@kangasta kangasta commented Apr 18, 2024

This also re-implements one resource, upcloud_network, and one data-source, upcloud_managed_object_storage_regions, using the plugin framework.

@kangasta kangasta force-pushed the refactor/plugin-framework-init branch 3 times, most recently from d957316 to c400264 Compare April 22, 2024 18:09
@kangasta kangasta marked this pull request as ready for review April 23, 2024 10:31
@kangasta kangasta requested a review from a team as a code owner April 23, 2024 10:31
@kangasta
Copy link
Contributor Author

License check found four new MPL-2.0 licenses, but those should be OK in this project.

@kangasta kangasta force-pushed the refactor/plugin-framework-init branch from e5f7751 to 3a5981e Compare April 23, 2024 11:41
@@ -39,17 +39,17 @@ resource "upcloud_router" "example_router" {

### Required

- `ip_network` (Block List, Min: 1, Max: 1) A list of IP subnets within the network (see [below for nested schema](#nestedblock--ip_network))
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I have understood right, the plugin framework does not mark any blocks required which would mark all blocks as optional in our docs 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found one related discussion answer

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Created issue to hashicorp/terraform-plugin-docs repository: hashicorp/terraform-plugin-docs#363

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Created also a PR in the hashicorp/terraform-plugin-docs repository: hashicorp/terraform-plugin-docs#366

docs/resources/network.md Outdated Show resolved Hide resolved
@kangasta kangasta merged commit 397b01e into main May 7, 2024
3 of 5 checks passed
@kangasta kangasta deleted the refactor/plugin-framework-init branch May 7, 2024 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants