Skip to content

Comments

Fix trail-snippet docs from PR #15 review#16

Merged
khaliqgant merged 2 commits intomainfrom
fix/trail-snippet-docs
Feb 20, 2026
Merged

Fix trail-snippet docs from PR #15 review#16
khaliqgant merged 2 commits intomainfrom
fix/trail-snippet-docs

Conversation

@khaliqgant
Copy link
Collaborator

@khaliqgant khaliqgant commented Feb 20, 2026

Summary

  • Remove pr-summary from export format list (CLI switch statement doesn't handle it, falls through to markdown)
  • Fix --branch example to show base branch (main) instead of feature branch — the CLI finds commits NOT in the specified branch
  • Fix --commits syntax from git range (abc123..def456) to comma-separated (abc123,def456) matching actual CLI parsing

Addresses review feedback from #15.

Test plan

  • Verify trail export --format docs match CLI switch cases
  • Verify trail compact --branch docs match git log <branch>..HEAD behavior
  • Verify trail compact --commits docs match comma-split parsing

🤖 Generated with Claude Code


Open with Devin

- Remove pr-summary from export format list (CLI doesn't implement it)
- Fix --branch example to show base branch (main) not feature branch
- Fix --commits syntax from range (abc..def) to comma-separated (abc,def)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@khaliqgant khaliqgant merged commit 27c1bb9 into main Feb 20, 2026
6 checks passed
@khaliqgant khaliqgant deleted the fix/trail-snippet-docs branch February 20, 2026 19:31
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