Skip to content

add cache to claude-anthropic#30

Merged
seveibar merged 2 commits intotscircuit:mainfrom
Anshgrover23:cache-anthropic
Jan 10, 2025
Merged

add cache to claude-anthropic#30
seveibar merged 2 commits intotscircuit:mainfrom
Anshgrover23:cache-anthropic

Conversation

@Anshgrover23
Copy link
Copy Markdown
Contributor

@Anshgrover23 Anshgrover23 commented Jan 10, 2025

Fixes #29
Fixes #25

Reference video:

2025-01-10.16-12-55.mov

one more video:

2025-01-10.16-11-41.mov

Comment thread lib/analyzePRWithClaude.ts
Copy link
Copy Markdown
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

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

fix the rethrow, then looks good, might need someone else to do another quick check

@seveibar seveibar merged commit a9dc017 into tscircuit:main Jan 10, 2025
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.

claude-pr-analysis differ everytime on generating readme. the tasks in the generate-weekly-overview should run concurrently

2 participants