Skip to content

activity section audit - #5302

Merged
jsundai merged 11 commits into
mainfrom
activity-sdk-review
Sep 18, 2026
Merged

jsundai merged 11 commits into
mainfrom
activity-sdk-review

Conversation

@jsundai

@jsundai jsundai commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

  • stray ([Temporal Docs][3]) citation in rust/activities/timeouts.mdx is gone
  • Activities tag is now present on all 41 pages under docs/develop/*/activities/ with the timeouts pages carrying an identical tag set across all eight SDK
  • "How to" heading cleanup

Notes to reviewers

@vercel

vercel Bot commented Sep 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
temporal-documentation Ready Ready Preview Sep 18, 2026 9:15pm UTC

Request Review

Comment thread docs/develop/dotnet/activities/timeouts.mdx Outdated
@jsundai

jsundai commented Sep 15, 2026

Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts in this pull request and resolve comments

# Conflicts:
#	docs/develop/dotnet/activities/execution.mdx
#	docs/develop/go/activities/basics.mdx
#	docs/develop/go/activities/execution.mdx
#	docs/develop/java/activities/execution.mdx
#	docs/develop/php/activities/basics.mdx
#	docs/develop/php/activities/execution.mdx
#	docs/develop/python/activities/basics.mdx
#	docs/develop/python/activities/execution.mdx
#	docs/develop/ruby/activities/execution.mdx
#	docs/develop/rust/activities/execution.mdx
#	docs/develop/typescript/activities/basics.mdx
#	docs/develop/typescript/activities/execution.mdx

Co-authored-by: jsundai <36107423+jsundai@users.noreply.github.com>

Copilot AI commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request and resolve comments

Done in 7953213. I merged main into this branch to resolve conflicts and reverted the two requested files (docs/develop/dotnet/activities/timeouts.mdx and docs/develop/java/activities/execution.mdx) back to their pre-bf2f303 state.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ jsundai
❌ Copilot
You have signed the CLA already but the status is still pending? Let us recheck it.

@jsundai
jsundai marked this pull request as ready for review September 15, 2026 19:49
@jsundai
jsundai requested a review from a team as a code owner September 15, 2026 19:49

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 994fcf41cd

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

description: This section explains how to set Activity Timeouts with the Java SDK
toc_max_heading_level: 4
tags:
- Activities

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Update the Java timeout description with its full scope

The audit updates the equivalent descriptions in docs/develop/{dotnet,go,php,python,ruby,rust,typescript}/activities/timeouts.mdx to mention the timeout types, Retry Policy, and next retry delay, but this Java page still says only that it explains how to set timeouts even though it also covers retry policies, next retry delay, and Heartbeats. Update this description alongside the other SDK pages so search results accurately represent the page.

AGENTS.md reference: AGENTS.md:L240-L243

Useful? React with 👍 / 👎.

Comment thread docs/develop/python/activities/execution.mdx Outdated
Keep the audit edits: Activity Execution capitalization in titles, sidebar
labels, and link labels; Define Activity parameters across basics pages;
restored Java execution anchors; and removal of the repeated ActivityOptions
paragraphs and the bold heading restatements.
Delete the page-announcing sentences on the asynchronous Activity pages and
the duplicate section list on the TypeScript timeouts page, relabel 'How to'
links and next-step bullets, and rewrite example lead-ins to state what the
example does.
@jsundai
jsundai enabled auto-merge (squash) September 18, 2026 21:02
@jsundai
jsundai merged commit d13800b into main Sep 18, 2026
14 of 15 checks passed
@jsundai
jsundai deleted the activity-sdk-review branch September 18, 2026 21:15
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.

4 participants