-
Notifications
You must be signed in to change notification settings - Fork 37
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
Remove SSH code from the API #13
Comments
This method is going to be removed from the API in the next major version and wasn't intended to be in the API in the first place. Prepares for addressing issue #13. Signed-off-by: aszlig <[email protected]>
I am a user of |
Ah, sorry, wanted to change the description of this issue. Didn't remember the exact implementation of them while writing this issue. So no, those methods won't be removed, sorry for the confusion. I'm however thinking about a new API architecture for 1.0.0, so the implementation of these methods will probably change. |
Maybe this warning can be removed then: |
@adrianog: Mhm, good point. What I had in mind back then was to move the function to another location. So maybe it's a good idea to move it already and change the warning to point to the new location. |
This really shouldn't belong to the API, so let's move it to the
hetznerctl
tool entirely.Scheduled for 1.0.0, because this is a backwards-incompatible change.
The text was updated successfully, but these errors were encountered: