Skip to content

Various enhancements to the GitHub Library to support our Temporal platform#468

Open
doggydogworld wants to merge 4 commits intomainfrom
gus/workflows-rerun-and-html-url
Open

Various enhancements to the GitHub Library to support our Temporal platform#468
doggydogworld wants to merge 4 commits intomainfrom
gus/workflows-rerun-and-html-url

Conversation

@doggydogworld
Copy link
Copy Markdown
Contributor

@doggydogworld doggydogworld commented Mar 31, 2026

This introduces four enhancements to the GitHub library that will be useful for our Temporal work.

  • Returning the HTML URL is useless for other libraries to rely on which is why it wasn't originally included. However it is good for enhancing user experience and for debugging when trying to track down a failed job.
  • Rerunning failed workflows has been added to allow for recovery of transient failures. Temporal can orchestrate this to recover from transient failures in workflows.
  • The GitHub App client now has a helper method to obtain the currently active OAuth token. This will be useful when our Temporal work needs to interact with Git directly (cloning teleport.e) which needs a valid token to authenticate with.
  • Duration is returned in WorkflowInfo. This will be used in our Temporal workflow to bubble up metrics related to GitHub Actions performance.

@doggydogworld doggydogworld requested a review from a team as a code owner March 31, 2026 19:16
@doggydogworld doggydogworld changed the title Adding rerun workflow and returning HTML URL for workflow dispatch Various enhancements to the GitHub Library to support our Temporal platform Apr 30, 2026
@aadc-dev
Copy link
Copy Markdown
Contributor

aadc-dev commented May 6, 2026

for posterity, the use of the clientID is recommended now: https://docs.github.com/en/apps/creating-github-apps/authenticating-with-a-github-app/generating-a-json-web-token-jwt-for-a-github-app

Currently the appID is used.

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.

3 participants