Skip to content

feat: add collapsible section to house the diff in the PR comment#63

Open
darrylb-github wants to merge 2 commits into
neondatabase:mainfrom
darrylb-github:feat/comment-diff-summary
Open

feat: add collapsible section to house the diff in the PR comment#63
darrylb-github wants to merge 2 commits into
neondatabase:mainfrom
darrylb-github:feat/comment-diff-summary

Conversation

@darrylb-github

Copy link
Copy Markdown

This PR adds a collapsible section to contain the schema diff in PR comments, improving readability by allowing users to expand/collapse the diff content as needed.

Changes

  • Wraps the diff content in HTML details and summary tags
  • Updates both the documentation example and the actual implementation in src/diff.ts
  • Maintains all existing functionality while improving UX

Benefits

  • Cleaner PR comment appearance
  • Users can choose when to view the full diff
  • Reduces visual clutter in PR conversations

@darrylb-github darrylb-github requested a review from a team as a code owner August 24, 2025 14:17

@orca-security-us orca-security-us Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@gsalomao gsalomao force-pushed the feat/comment-diff-summary branch 2 times, most recently from 9bdd29b to e6af9c1 Compare September 25, 2025 08:13
@gsalomao gsalomao force-pushed the feat/comment-diff-summary branch from e6af9c1 to 6d3061e Compare September 25, 2025 08:50
@gsalomao

Copy link
Copy Markdown
Contributor

@darrylb-github, could you please run bun run format:write to fix formatting issues?

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.

2 participants