[BUG]: WorkflowJobStatus enum does not contain "pending" value. about octokit.net HOT 4 CLOSED

Thanks for the detailed explanation! I've done a little code spelunking internally and I think it's appropriate to add a Pending value here.

from octokit.net.

@kfcampbell should the existing enum just be extended with a Pending member, or is the fix to split this into a separate enum WorkflowJobStepStatus? I didn't see anything int he REST API docs what the legal values for this enum were, and it already has a Waiting which seems like it would be the same as Pending?

from octokit.net.

@redoz I'm sorry I'm a bit confused. Your title mentions WorkflowJobStatus but the description mentions WorkflowJobStep. Do you mind linking to the documentation you're looking at here?

from octokit.net.

@kfcampbell Sorry for the confusion, I'm getting the error while accessing the Status property on the WorkflowJobStep, that property is of type StringEnum<WorkflowJobStatus> which is the same type of the Status property on the WorkflowJob type. It wasn't clear if they are referring to the same status (of the WorkflowJob), or if the enum is just re-used to represent the state of the WorkflowJobStep.

I can easily submit a PR to add the Pending value to this enum, but I wasn't sure if this was the correct fix, or if there is more to this than it seems, given that the enum already has a Waiting member.

And maybe I'm looking at the wrong place but it seems the step is documented to have the following statuses:

based on this documentation: https://docs.github.com/en/rest/actions/workflow-jobs?apiVersion=2022-11-28

So, before I put effort into submitting a PR, I figured I'd ask for some clarification to make sure it got fixed correctly. I hope that makes more sense, thanks for your time so far!

from octokit.net.