Skip to content
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

Correct debug cypress:server:browsers selection #5929

Merged
merged 2 commits into from
Sep 18, 2024

Conversation

MikeMcC399
Copy link
Contributor

Issue

The example DEBUG value cypress:server:browsers:* is incomplete.

Debug output for cypress:server:browsers is not logged in this case.

The debug module does not support any hierarchy based on colon characters :. This is a simple string match.

Change

To also log debug output from packages/server/lib/browsers/index.ts which defines a debug module name cypress:server:browsers, the environment value is changed to

cypress:server:browsers*

removing the colon character :.

Note

  • This was an error in the correction made by PR docs: rework print debug logs #5921, which added the sub-modules of cypress:server:browsers and accidentally left out the top-level in the correction process.

Copy link

netlify bot commented Sep 18, 2024

👷 Deploy request for cypress-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 205f176

@cypress-app-bot
Copy link
Collaborator

@MikeMcC399 MikeMcC399 marked this pull request as ready for review September 18, 2024 15:02
@jennifer-shehane jennifer-shehane merged commit 05cea06 into cypress-io:main Sep 18, 2024
3 checks passed
@MikeMcC399 MikeMcC399 deleted the debug-browsers branch September 19, 2024 06:34
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