Skip to content

add retry logic to the action#5

Merged
waheedahmed merged 1 commit into
mainfrom
waheed/add-retry-logic
Apr 30, 2026
Merged

add retry logic to the action#5
waheedahmed merged 1 commit into
mainfrom
waheed/add-retry-logic

Conversation

@waheedahmed
Copy link
Copy Markdown
Owner

@waheedahmed waheedahmed commented Apr 30, 2026

This pull request adds configurable retry logic for enqueuing pull requests, allowing users to specify how many times to retry and how long to wait between retries if the initial enqueue attempt fails. It also updates documentation and input defaults to reflect these new options.

Enqueue Retry Logic Enhancements:

  • Added enqueue-retries and enqueue-retry-sleep inputs to allow configuration of retry attempts and sleep duration between retries when enqueuing a PR fails. Defaults are 6 retries and 5000ms sleep. Setting either to 0 disables the respective behavior. (action.yml, src/api.js, README.md) [1] [2] [3] [4] [5] [6]
  • Implemented the retry and sleep logic in the enqueue process, with clear logging and failure handling. (src/api.js) [1] [2]

Documentation and Input Updates:

  • Updated README.md and input tables to document the new retry options, including usage examples. [1] [2]
  • Updated test configuration to include new inputs with default values. (src/__tests__/index.test.js)
  • Clarified example skip label in action.yml.

Other Updates:

  • Bumped Node.js version from 20 to 22 in action.yml and updated package version to 0.0.5. (action.yml, package.json) [1] [2]

@waheedahmed waheedahmed force-pushed the waheed/add-retry-logic branch from 9dd7f36 to 83872e6 Compare April 30, 2026 15:10
@waheedahmed waheedahmed force-pushed the waheed/add-retry-logic branch from 83872e6 to 269018a Compare April 30, 2026 15:14
@waheedahmed waheedahmed merged commit 464a313 into main Apr 30, 2026
3 checks passed
@waheedahmed waheedahmed deleted the waheed/add-retry-logic branch April 30, 2026 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant