Skip to content

fix(miners): case-insensitive joins for review_summary and repos#185

Open
kiannidev wants to merge 1 commit into
entrius:testfrom
kiannidev:fix/183-miners-case-insensitive-joins
Open

fix(miners): case-insensitive joins for review_summary and repos#185
kiannidev wants to merge 1 commit into
entrius:testfrom
kiannidev:fix/183-miners-case-insensitive-joins

Conversation

@kiannidev

@kiannidev kiannidev commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Match pr_review_summary and repos joins on LOWER(repo_full_name) in miner pull request queries
  • Fixes missing review_summary and default_branch for mixed-case repo PKs, especially in getPullRequestsByRepo

Related Issues

Fixes #183

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other (describe below)

Testing

  • npm run build in packages/das passes

Checklist

  • I have read the Contributing Guide
  • Code builds without errors
  • New and existing tests pass (if applicable)
  • Documentation updated (if applicable)
  • No unnecessary dependencies added

@xiao-xiao-mao xiao-xiao-mao Bot added the bug Something isn't working label Jun 15, 2026
@kiannidev kiannidev force-pushed the fix/183-miners-case-insensitive-joins branch from 618941d to 741bf96 Compare June 17, 2026 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] miners API: getPullRequestsByRepo joins miss review_summary for mixed-case repos

1 participant