Skip to content

Hide visible merge conflict markers in PR review#154

Merged
BunsDev merged 3 commits into
mainfrom
okcode/hide-merge-conflict-error
Apr 1, 2026
Merged

Hide visible merge conflict markers in PR review#154
BunsDev merged 3 commits into
mainfrom
okcode/hide-merge-conflict-error

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented Apr 1, 2026

Summary

  • Fall back to reading conflicted file stages directly from the git index when conflict markers are missing or unparsable.
  • Allow merge conflict resolutions to write full-file candidates when the source came from index-based fallback.
  • Improve the skills detail dialog layout with an icon header, tighter tag styling, and clearer command/path blocks.
  • Make dialog panels and the skills page content area scroll correctly with long content.

Testing

  • Not run (not requested).
  • Not run: bun fmt.
  • Not run: bun lint.
  • Not run: bun typecheck.

BunsDev added 2 commits March 31, 2026 23:52
- Add icon, tag chips, and improved command/path presentation
- Make dialog content and skills page scrollable within available height
- Fall back to ours/theirs content from the git index when conflict markers cannot be parsed
- Apply full-file fallback candidates directly during resolution
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 1, 2026 5:18am

@github-actions github-actions Bot added size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Apr 1, 2026
- Replace the beta range with the stable 1.1.8 release
- Update the lockfile to match the pinned dependency
@BunsDev BunsDev merged commit f5dc652 into main Apr 1, 2026
7 of 9 checks passed
@BunsDev BunsDev deleted the okcode/hide-merge-conflict-error branch April 1, 2026 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant