Skip to content

fix(forge): show expand row after last hunk in PR review#426

Merged
agavra merged 1 commit into
agavra:mainfrom
saleemdjima:fix/missing-expand-after-last-hunk-423
Jun 22, 2026
Merged

fix(forge): show expand row after last hunk in PR review#426
agavra merged 1 commit into
agavra:mainfrom
saleemdjima:fix/missing-expand-after-last-hunk-423

Conversation

@saleemdjima

Copy link
Copy Markdown
Contributor

This PR fixes #423

Summary

  • Fix the missing expand row after the last hunk in PR review mode
  • Added file_line_count to ForgeBackend so the forge API is used directly to count lines
  • Fixed slice_to_diff_lines in both forge backends to break early instead of iterating up

@agavra agavra left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM and tried it out locally. Thanks @saleemdjima :)

@agavra
agavra merged commit efa05f6 into agavra:main Jun 22, 2026
4 checks passed
joaomendoncaa pushed a commit to joaomendoncaa/tuicr that referenced this pull request Jun 26, 2026
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.

Expand down doesn't show after last hunk

2 participants