Skip to content

fix(pr-evaluator): exclude lock files and setup reports from evaluation#328

Merged
edwinyjlim merged 2 commits intomainfrom
pr-eval-ignore
Feb 11, 2026
Merged

fix(pr-evaluator): exclude lock files and setup reports from evaluation#328
edwinyjlim merged 2 commits intomainfrom
pr-eval-ignore

Conversation

@edwinyjlim
Copy link
Member

  • Add exclusion patterns for all common lock files (pnpm, npm, yarn, bun, etc.)
  • Exclude wizard/posthog setup report files from diff evaluation
  • Prevents 'prompt too long' errors when evaluating PRs with large lock file changes

edwinyjlim and others added 2 commits February 11, 2026 13:54
- Add exclusion patterns for all common lock files (pnpm, npm, yarn, bun, etc.)
- Exclude wizard/posthog setup report files from diff evaluation
- Prevents 'prompt too long' errors when evaluating PRs with large lock file changes

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Add Podfile.lock (CocoaPods/React Native iOS)
- Add pubspec.lock (Flutter/Dart)
- Add *.lockfile (Gradle dependency locks)
- Add packages.lock.json (NuGet/Xamarin/.NET MAUI)
- Organize exclusions by language/platform for better maintainability

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@edwinyjlim edwinyjlim merged commit 2526897 into main Feb 11, 2026
7 checks passed
@edwinyjlim edwinyjlim deleted the pr-eval-ignore branch February 11, 2026 19:39
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