Skip to content

Cache Github event details - #98

Merged
f-ewald merged 3 commits into
mainfrom
feature/github-cache
Jan 12, 2026
Merged

Cache Github event details#98
f-ewald merged 3 commits into
mainfrom
feature/github-cache

Conversation

@f-ewald

@f-ewald f-ewald commented Jan 12, 2026

Copy link
Copy Markdown
Owner

This change caches Github event details for the duration the run to improve the runtime and speed up duplicate calls as well as save API quota.

This change caches Github event details for the duration the run to improve the runtime and speed up duplicate calls as well as save API quota.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds caching for GitHub pull request details to reduce redundant API calls and improve performance during event fetching. The cache stores PR details by repository and PR number, reusing them when the same PR appears multiple times in the event stream.

Changes:

  • Added an @event_cache hash to store pull request details by repository and PR number
  • Modified create_pull_request_event and create_pull_request_review_event to use cached PR details when available
  • Added a cache_key helper method to generate consistent cache keys

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lib/github/client.rb
Comment thread lib/github/client.rb
Comment thread lib/github/client.rb Outdated
Comment thread lib/github/client.rb
Comment thread lib/github/client.rb
@github-actions

Copy link
Copy Markdown
Contributor

Code Coverage

Package Line Rate Complexity Health
worklog 79% 0
Summary 79% (743 / 944) 0

@f-ewald
f-ewald merged commit 84d961a into main Jan 12, 2026
1 check passed
@f-ewald
f-ewald deleted the feature/github-cache branch January 12, 2026 19:52
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.

2 participants