-
Notifications
You must be signed in to change notification settings - Fork 224
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
add temporal cli docs for nexus endpoints #3149
base: main
Are you sure you want to change the base?
Conversation
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.
Assuming this is all autogenerated and there's nothing critical to review.
Signed-off-by: Phil Prasek <[email protected]>
Signed-off-by: Phil Prasek <[email protected]>
Signed-off-by: Phil Prasek <[email protected]>
76a13e0
to
44bdcb0
Compare
docs/cli/cmd-options.mdx
Outdated
@@ -313,7 +317,11 @@ Use `-1` to fetch Child Workflows at any depth. | |||
|
|||
## description | |||
|
|||
Namespace description. | |||
Namespace description or Endpoint description in markdown format (encoded using the configured codec server). |
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.
codec server -> Codec Server
@@ -656,6 +672,18 @@ Specify a delay before the workflow starts. | |||
|
|||
Backfill start time. | |||
|
|||
## target-namespace | |||
|
|||
Namespace in which a handler worker will be polling for Nexus tasks on. |
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.
Great job & looks good to me! Just make sure all the Temporal terms are capitalized like "Worker", "Workflow", "Task Queue" and make sure every sentence is on one line. I did the Operator file as an example.
- [--tls-server-name](/cli/cmd-options#tls-server-name) | ||
|
||
- [--unset-description](/cli/cmd-options#unset-description) | ||
|
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.
@fairlydurable Do you mind giving one quick look when you get back from PTO since the CLI is your realm of expertise? If there's anything major we can all probably hop on a call about it.
Signed-off-by: Phil Prasek <[email protected]>
What does this PR do?
Adds temporal cli docs for nexus endpoints
Notes to reviewers
Auto-generated using a fork of the recent docs gen.