Skip to content
New issue

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

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

Already on GitHub? # to your account

Docs: Remove jestPlaywrightDebug ? #636

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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants