Skip to content

Docs: Remove jestPlaywrightDebug ? #636

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
thernstig opened this issue Mar 16, 2021 · 2 comments
Closed

Docs: Remove jestPlaywrightDebug ? #636

thernstig opened this issue Mar 16, 2021 · 2 comments

Comments

@thernstig
Copy link
Contributor

https://github.com/playwright-community/jest-playwright#jestplaywrightdebug mentions the use of jestPlaywrightDebug. But with the support for PWDEBUG=1 jest is jestPlaywrightDebug needed anymore? If not it should be removed from the docs and code.

Sorry for not using the template but it feels way too elaborate for something like this that exists in the README.md 😝

@mmarkelov
Copy link
Member

@thernstig this is helper function to simplify debugging process. But this is not related to jestPlaywright.debug() that we removed.

@thernstig
Copy link
Contributor Author

I was aware, but I just did not see the reason for test.jestPlaywrightDebug. One can easily do if (process.env.PWDEBUG) in jest-playwright.config.js. I thought it might simplify some logic for you and you could just describe how to use that env var.

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

No branches or pull requests

2 participants