jaredpalmer / cypress-image-snapshot

Catch visual regressions in Cypress
MIT License
887 stars 160 forks source link

[Feature request] Integrate odiff instead of pixelmatch #174

Open dmtrKovalenko opened 3 years ago

dmtrKovalenko commented 3 years ago

Hey, I build odiff exactly to be used for cypress visual regression, according to an ability to run screenshot diffs from paths directly it is faster than pixel match by 6-7 times usually.

I'd like to make a PR that integrates odiff as an option to be used directly instead of jest-image-snapshot. This will basically allow removing the huge amount of code and will speed up the comparison.

What do you think? Any kind of blockers or reasons why the PR might be rejected, should I build the alternative library instead?