[Feature Request] Remove RetryState.RETRY_STATE_IN_PROGRESS from public api

Open
#232 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
35/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
grpc
Domain
api

Research direction

The enum declaration is in temporal/api/enums/v1/workflow.proto around line 107. Start by reviewing RetryState.RETRY_STATE_IN_PROGRESS and the API compatibility expectations; done means this internally used state is no longer exposed through the public API.

Written by the indexing model from the issue text.

Description

enhancement

RetryState.RETRY_STATE_IN_PROGRESS
is used internally only but is exposed via public api: https://github.com/temporalio/api/blob/master/temporal/api/enums/v1/workflow.proto#L107

This is causing some confusions to our users. If possible please remove it (keep it internal only).

Idk if this should be issue opened on the server or here. Please let me know if this repo is wrong place for it.

Dominant language
Go
Stars
133
Forks
101
Avg merge
1d 22h
Merged PRs (30d)
11

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from temporalio/api

All issues in temporalio/api

Similar issues

More Go issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.