-
Notifications
You must be signed in to change notification settings - Fork 6
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
Service endpoint -> Deployments #229
Conversation
Deploying with Cloudflare Pages
|
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.
Thanks for creating this PR @slinkydeveloper. The changes look good to me. One question that came up was whether we should rather use "service deployment" instead of "deployment" alone in the documentation texts. Maybe this makes it clearer that we are talking about service deployment and not a server deployment?
I think you're right on this. let me go through this pr again then. |
uhhhhh if we change to |
I did another pass at this, sometimes using the term Service deployment |
Co-authored-by: Till Rohrmann <[email protected]>
fb53c97
to
a2407b3
Compare
(not everywhere)
a2407b3
to
b35a27c
Compare
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.
LGTM. Thanks for updating the PR @slinkydeveloper. +1 for merging.
Co-authored-by: Till Rohrmann <[email protected]>
See restatedev/restate#989