feat: gh run rerun|cancel

[GustavEikaas]

Issue: #449

• Update readme

Details buffer

• Rerun
• Rerun only failed jobs
• Cancel

Telescope

• Rerun
• Rerun only failed jobs
• Cancel

Keybindings Update

Added the following keybindings to buffer and telescope:

• <C-o> → Rerun workflow
• <C-f> → Rerun failed workflow
• <C-x> → Cancel workflow

[williambdean]

Thanks for the PR! Looks awesome! 🎉

How easy is it to make a Octo run rerun command? Do the pickers support callbacks on the selected item? Thinking something similar to Octo issue list and Octo issue develop

[GustavEikaas]

You mean Octo run rerun like in?

1. :Octo run rerun
2. Telescope picker listing all wf runs
3. Press enter to trigger a rerun
4. Go to details buffer of wf run

[williambdean]

Yeah, exactly. Would that be useful as well?

[GustavEikaas]

I'm not convinced tbh, I'm struggling to see the use-case.

The way I see it, the user flow would be something like this

1. User triggers a workflow
2. User notices the workflow has failed
3. User modifies and external system before rerunning the workflow (or the workflow is flaky af)
4. User triggers a rerun of the workflow

So to me I dont really understand how the user would know that the workflow failed. There are multiple cases I guess

1. :Octo run list reveals that the workflow failed -> use telescope binding to rerun?
2. The octo workflow run buffer reveals that the workflow failed -> use buffer binding to rerun
3. Github UI shows a failed workflow -> easily retrigger from github ui

I dont mind implementing it, just personally I dont see what the user flow is

EDIT

I guess there are other motivations for rerunning a workflow rather than failed workflow. e.g workflow_dispatch etc
But I would still want to see the details of old run before rerunning

[GustavEikaas]

Okay so basically, I have added keybindings for rerun | cancel in both telescope and buffer. I could need some good suggestions on which sensible keybindings to use

[williambdean]

Maybe <C-x> for cancel?

[GustavEikaas]

@williambdean What do you think about merging this PR, which adds cancel | rerun | rerun failed to telescope and octo wf buffer.

If we want Octo run cancel and Octo run rerun it could be a separate pr?

How easy is it to make a Octo run rerun command? Do the pickers support callbacks on the selected item? Thinking something similar to Octo issue list and Octo issue develop

Not that hard to add but I think it merits some discussion about how the signature/syntax should be

[williambdean]

Yeah, let's wait on that then!

Did you want to update the README still?

[GustavEikaas]

Readme updated✅
Some tests are failing but it seems unrelated to my changes?

[williambdean]

Yeah, they seem unrelated. Let me take a look at the changes

[GustavEikaas]

@williambdean It's good to go I think

[williambdean]

Great. Taking a look

[williambdean]

Works great. Thanks!