Skip to content

Conversation

yuandrew
Copy link
Contributor

@yuandrew yuandrew commented Oct 15, 2025

What was changed

Replaced all tctl references with Temporal CLI.
Removed search attribute tctl docs link because it's a broken link

Why?

tctl deprecated

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

Note

Replaces tctl mentions with Temporal CLI across docs/Javadocs, updates links/wording, and removes broken search-attribute doc links.

  • Docs/Javadocs updates:
    • Update README.md and temporal-remote-data-encoder/README.md to reference Temporal CLI (links and wording).
    • Replace "tctl" with "Temporal CLI" in Javadocs for:
      • io.temporal.common.WorkflowExecutionHistory#fromJson param docs.
      • io.temporal.common.converter.DefaultDataConverter description.
      • io.temporal.internal.common.HistoryJsonUtils class Javadoc.
    • Remove broken "tctl search attribute" doc links from testing APIs:
      • temporal-testing/.../TestEnvironmentOptions.java
      • temporal-testing/.../TestWorkflowEnvironment.java
      • temporal-testing/.../TestWorkflowExtension.java
      • temporal-testing/.../TestWorkflowRule.java

Written by Cursor Bugbot for commit 19de93e. This will update automatically on new commits. Configure here.

@yuandrew yuandrew requested a review from a team as a code owner October 15, 2025 23:54
* @param name name of the search attribute
* @param type search attribute type
* @return {@code this}
* @see <a
Copy link
Member

Choose a reason for hiding this comment

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

Do we have equivalent links for CLI we can sub here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's a broken link. I didn't find any direct link we can use to replace, but I guess I could link to the generic https://docs.temporal.io/search-attribute page, which has a point on using the CLI at the bottom of the site

Copy link
Member

Choose a reason for hiding this comment

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

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.

2 participants