Skip to content

docs: align Worker Options with the API - #500

Open
yhwang-nydig wants to merge 2 commits into
temporalio:mainfrom
yhwang-nydig:docs/concepts-worker-options-match-api
Open

docs: align Worker Options with the API#500
yhwang-nydig wants to merge 2 commits into
temporalio:mainfrom
yhwang-nydig:docs/concepts-worker-options-match-api

Conversation

@yhwang-nydig

Copy link
Copy Markdown
Contributor

The Worker Options section documented a deploymentName field that does not exist. This replaces it with unsafeCustomBuildID, includes its safety caveat, and clarifies how the Temporal deployment name is derived.

The Worker Options section documented a deploymentName field that does not
exist in WorkerOptions (the Temporal deployment name is always derived from
the Kubernetes namespace and WorkerDeployment name, and is not
user-configurable), while omitting the unsafeCustomBuildID field that does
exist.

Replace the phantom field with the real one — including its workflow-safety
caveat inline — and state the deployment-name derivation explicitly,
cross-linking the TEMPORAL_DEPLOYMENT_NAME section that already describes it.
@yhwang-nydig
yhwang-nydig requested review from a team, eniko-dif and jlegrone as code owners August 5, 2026 18:48
@CLAassistant

CLAassistant commented Aug 5, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@jaypipes jaypipes left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just aligns the docs, cool with me. thank you @yhwang-nydig

@jaypipes
jaypipes enabled auto-merge (squash) August 6, 2026 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants