-
Notifications
You must be signed in to change notification settings - Fork 182
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
Update Go Guide #2250
Update Go Guide #2250
Conversation
I need a Go expert to let me know if we should bump up the minimum version in the pre-requisites. |
@megglos I added you, but feel free to delegate to a Zeebe/Go expert. Most of the changes I made were incorporating both Modeler experiences and deploying your model from a file on your desktop programmatically or by clicking the deploy button in Web Modeler. |
Added @Sijoma for his Go expertise, but he's on leave right now. (Also, stay on leave, this is not urgent!) |
Hey @abbasadel given you recently worked on the go client update, could you take a look at this? |
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.
This is so CLEAN! Approving from a technical perspective.
I added @abbasadel's changes. Can I get a 👍 review so we can merge this? |
Description
Closes #773.
When should this change go live?
hold
label or convert to draft PR)?PR Checklist
/versioned_docs
directory, or they are not for an already released version./docs
directory (aka/next/
), or they are not for future versions.