Skip to content

add diff_pixels, diff_ratio for acceptence, similar as playwright node.js version#4

Open
hgye wants to merge 16 commits into
kumaraditya303:mainfrom
hgye:main
Open

add diff_pixels, diff_ratio for acceptence, similar as playwright node.js version#4
hgye wants to merge 16 commits into
kumaraditya303:mainfrom
hgye:main

Conversation

@hgye
Copy link
Copy Markdown

@hgye hgye commented May 18, 2022

No description provided.

@iloveitaly
Copy link
Copy Markdown

This looks great, what do you think @kumaraditya303

Comment thread setup.py
author_email="",
description="A pytest wrapper for snapshot testing with playwright",
name="pytest-playwright-visual",
author="Symon Storozhenko",
Copy link
Copy Markdown
Owner

@kumaraditya303 kumaraditya303 Dec 12, 2024

Choose a reason for hiding this comment

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

You shouldn't be changing the author nor the project name.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@hgye are you open to changing this so we can get this merged in?

@iloveitaly
Copy link
Copy Markdown

Decided to fork and implement these changes and a bunch more: https://github.com/iloveitaly/pytest-playwright-visual-snapshot

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.

3 participants