-
Notifications
You must be signed in to change notification settings - Fork 43
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
Testing out the beta Cloudflare SDK #720
Comments
@jacobbednarz I will have time working on this after finishing up #797. Sorry about the loooooong delay! |
all good, no rush at all! |
@jacobbednarz I'm just writing down my own notes for my self. There's no new information. Observed Improvements of the New SDK
BlockerThe only blocker seems to be the lack of the ability to easily create a "clean" client from the new API. This can be worked around by undoing the reading of environment variables. The above improvements might eventually convince me to adopt the new SDK with such a workaround. |
thanks for kicking the tyres here and keeping us updated ❤️
there is currently a one hour (maximum) delay. they are built and triggered from the same pipeline. |
👋 thanks for keeping this project going! it's been a great source of improvements for the cloudflare-go library and as a by product, the Terraform Provider.
we've been working on a beta of the cloudflare-go library (cloudflare/cloudflare-go#1538) that is generated from our OpenAPI schemas and if you've got some free time, i'd love you to kick the tyres on it and let us know if you have any feedback! (no pressure of course). we have a migration guide included but you're also welcome to just eyeball it and see if it all feels like it should.
if you do find anything that doesn't work nicely for your use case, or just have some design/library wide questions, please don't hesitate to drop them into the linked discussion.
cheers!
The text was updated successfully, but these errors were encountered: