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: move data.hcloud_ssh_key(s) to Plugin Framework #817

Merged
merged 1 commit into from
Mar 11, 2024

Conversation

jooola
Copy link
Member

@jooola jooola commented Dec 12, 2023

Related to #752

@jooola jooola force-pushed the plugin-framework-ssh-keys branch from 3515bec to 3a685a3 Compare December 19, 2023 15:01
@apricote apricote force-pushed the plugin-framework-ssh-keys branch from 3a685a3 to f664222 Compare January 22, 2024 10:12
Copy link

codecov bot commented Jan 22, 2024

Codecov Report

Attention: Patch coverage is 73.83178% with 84 lines in your changes are missing coverage. Please review.

Project coverage is 69.41%. Comparing base (7d8bf6e) to head (12be994).

Files Patch % Lines
internal/sshkey/data_source.go 67.22% 32 Missing and 7 partials ⚠️
internal/sshkey/resource.go 66.97% 23 Missing and 13 partials ⚠️
internal/sshkey/data_source_list.go 88.60% 5 Missing and 4 partials ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #817       +/-   ##
===========================================
+ Coverage   33.82%   69.41%   +35.59%     
===========================================
  Files          54       56        +2     
  Lines        7649     7732       +83     
===========================================
+ Hits         2587     5367     +2780     
+ Misses       5044     1699     -3345     
- Partials       18      666      +648     
Flag Coverage Δ
e2e 69.41% <73.83%> (?)
unit 33.71% <28.03%> (-0.11%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jooola jooola force-pushed the plugin-framework-ssh-keys branch from 8bc9006 to 5ae39c3 Compare January 23, 2024 14:07
@jooola jooola force-pushed the plugin-framework-ssh-keys branch 5 times, most recently from 323e010 to 6715438 Compare February 2, 2024 15:05
@jooola jooola force-pushed the plugin-framework-ssh-keys branch from 6715438 to 1411652 Compare February 5, 2024 08:23
apricote added a commit that referenced this pull request Feb 26, 2024
apricote added a commit that referenced this pull request Feb 26, 2024
apricote added a commit that referenced this pull request Feb 26, 2024
All current resources use string IDs, but they are ints in hcloud-go.
This util helps convert between them.

Will be used by #817
@apricote apricote force-pushed the plugin-framework-ssh-keys branch from f6ae3c3 to 5e7878f Compare February 26, 2024 09:51
@jooola jooola marked this pull request as ready for review February 26, 2024 09:52
@jooola jooola requested a review from a team as a code owner February 26, 2024 09:52
@apricote apricote force-pushed the plugin-framework-ssh-keys branch 2 times, most recently from cbace55 to 8579b2c Compare February 26, 2024 10:35
apricote added a commit that referenced this pull request Mar 11, 2024
We often have to convert between the Terraform Plugin Framework and
hcloud-go representations for labels.

Will be used by #817
@apricote apricote force-pushed the plugin-framework-ssh-keys branch from 8579b2c to 12be994 Compare March 11, 2024 11:10
@apricote apricote merged commit 3bc9b13 into main Mar 11, 2024
6 checks passed
@apricote apricote deleted the plugin-framework-ssh-keys branch March 11, 2024 12:41
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