Skip to content

[#761]: feat(protocol): ensure LocalActivityParams includes required SDK fields#761

Merged
rustatian merged 2 commits into
masterfrom
local-activity
May 5, 2026
Merged

[#761]: feat(protocol): ensure LocalActivityParams includes required SDK fields#761
rustatian merged 2 commits into
masterfrom
local-activity

Conversation

@xepozz
Copy link
Copy Markdown
Collaborator

@xepozz xepozz commented May 5, 2026

  • Added DataConverter, FailureConverter, and Attempt fields to LocalActivityParams.
  • Avoided potential nil-pointer dereference in Go SDK.
  • Introduced unit tests to validate required field population and default values.

Reason for This PR

[Author TODO: add issue # or explain reasoning.]

Description of Changes

[Author TODO: add description of changes.]

License Acceptance

By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.

PR Checklist

[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

  • All commits in this PR are signed (git commit -s).
  • The reason for this PR is clearly provided (issue no. or explanation).
  • The description of changes is clear and encompassing.
  • Any required documentation changes (code and docs) are included in this PR.
  • Any user-facing changes are mentioned in CHANGELOG.md.
  • All added/changed functionality is tested.

@xepozz xepozz requested a review from rustatian as a code owner May 5, 2026 11:05
@rustatian rustatian added the C-enhancement Category: enhancement. Meaning improvements of current module, transport, etc.. label May 5, 2026
- Added `DataConverter`, `FailureConverter`, and `Attempt` fields to `LocalActivityParams`.
- Avoided potential nil-pointer dereference in Go SDK.
- Introduced unit tests to validate required field population and default values.
@xepozz xepozz force-pushed the local-activity branch from f9cdd24 to 797fbbb Compare May 5, 2026 11:35
Copy link
Copy Markdown
Collaborator

@rustatian rustatian left a comment

Choose a reason for hiding this comment

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

LGTM, thank you @xepozz 👍🏻

@rustatian rustatian changed the title feat(protocol): ensure LocalActivityParams includes required SDK fields [#761]: feat(protocol): ensure LocalActivityParams includes required SDK fields May 5, 2026
@rustatian rustatian merged commit af58a82 into master May 5, 2026
11 checks passed
@rustatian rustatian deleted the local-activity branch May 5, 2026 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-enhancement Category: enhancement. Meaning improvements of current module, transport, etc..

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants