Skip to content

Add --skip-compare-text option#19

Merged
JustusRijke merged 1 commit intomainfrom
feat/skip-compare-text
Mar 1, 2026
Merged

Add --skip-compare-text option#19
JustusRijke merged 1 commit intomainfrom
feat/skip-compare-text

Conversation

@JustusRijke
Copy link
Owner

Summary

  • Adds --skip-compare-text CLI flag and skip_compare_text API parameter to bypass the text content comparison stage
  • Useful when different PDF render engines produce identical-looking PDFs with different text extraction order, which would otherwise cause false positive failures

Test plan

  • test_skip_compare_text — verifies skipping text stage still catches visual differences
  • test_skip_compare_text_multiplatform — verifies multiplatform PDFs pass with flag enabled
  • test_skip_compare_text_flag — verifies CLI flag wires through correctly

@codecov
Copy link

codecov bot commented Mar 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@JustusRijke JustusRijke merged commit 3763f92 into main Mar 1, 2026
9 checks passed
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