Skip to content

Add Retry option to Add-to-project action #700

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

zwick
Copy link

@zwick zwick commented Jun 11, 2025

Context

Users of this action have reported failing tasks due to a race condition with this action. The above linked PR suggested adding a retry capability to the Octokit call. A plugin exists for Octokit to retry failed requests utilizing a standard Backoff and was leveraged for this functionality.

Changes

  • Installed and imported octokit-retry package to action
  • Automatically retry failed requests in the Add-to-project action

@zwick zwick self-assigned this Jun 11, 2025
@zwick zwick added enhancement New feature or request dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels Jun 11, 2025
@zwick zwick force-pushed the zwick/add-retries branch from b7befbf to 095d3fc Compare June 11, 2025 14:32
@skw
Copy link
Contributor

skw commented Jul 7, 2025

@zwick Were you planning to get this PR ready for review? I think you may need to update the package.json and lock file.

@zwick zwick force-pushed the zwick/add-retries branch from 095d3fc to 8621a20 Compare July 8, 2025 14:22
- Imported octokit-retry package to action
@zwick zwick force-pushed the zwick/add-retries branch from 8621a20 to 3b6cd34 Compare July 8, 2025 14:26
@zwick
Copy link
Author

zwick commented Jul 8, 2025

@skw My efforts are a bit stuck at the moment. The failures in these tests are accurate, we are having some Typescript compatibilities between the Retry plugin and the version of Octokit Core bundled with @actions/github.

I'm currently down a bit of a rabbit hole while attempting to leverage @octokit/core instead. Importing this requires rewriting the action to an ES Module instead of targeting commonjs. However I am having issues with successfully importing Octokit Core after the change to a module.

This won't be ready until this can be solved, but I'm trying to not spend an exceeding amount of time on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants