Skip to content

Conversation

@twaugh
Copy link
Owner

@twaugh twaugh commented Oct 17, 2025

No description provided.

…tion

Fix several issues with grepdiff documentation in patchutils.xml:

* Move misplaced grepdiff example from filterdiff Examples section to
  grepdiff Examples section. The advanced example showing
  --output-matching=hunk with --as-numbered-lines was incorrectly
  placed in filterdiff documentation.

* Clean up grepdiff Description section by removing embedded example
  that used a complex, inefficient pipeline (grepdiff | xargs filterdiff).
  Keep the section focused on conceptual explanation.

* Completely rewrite grepdiff Examples section with comprehensive,
  progressive examples:
  - Start with filename-only output showing why it's useful for
    quick patch scanning
  - Show --output-matching=hunk for seeing actual matching hunks
  - Show --output-matching=file for complete file diffs
  - Include advanced usage with line numbering options
  - Preserve Git format example with explanatory notes

* Add explanatory text for each example showing when and why to use
  each mode, making grepdiff's capabilities clearer

These changes better showcase grepdiff as a powerful standalone tool
rather than just a helper for filterdiff, with clear examples of its
different output modes and proper organization of documentation.

Assisted-by: Cursor
@codecov
Copy link

codecov bot commented Oct 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.50%. Comparing base (551a06c) to head (67bb648).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #146   +/-   ##
=======================================
  Coverage   83.50%   83.50%           
=======================================
  Files           5        5           
  Lines        4159     4159           
  Branches      985      985           
=======================================
  Hits         3473     3473           
  Misses        686      686           
Flag Coverage Δ
unittests 83.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@twaugh twaugh merged commit ce2a79b into master Oct 17, 2025
7 checks passed
@twaugh twaugh deleted the docs branch October 17, 2025 14:43
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