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

docs: add DEBUG=corepack logging instructions #605

Merged
merged 2 commits into from
Jan 26, 2025

Conversation

MikeMcC399
Copy link
Contributor

@MikeMcC399 MikeMcC399 commented Jan 21, 2025

Change

Add information to the following documents regarding use of the npm debug module by setting the related environment variable DEBUG=corepack to enable additional debug logging:

@MikeMcC399 MikeMcC399 marked this pull request as ready for review January 21, 2025 08:57
CONTRIBUTING.md Outdated Show resolved Hide resolved
@MikeMcC399 MikeMcC399 requested a review from aduh95 January 24, 2025 14:34
@aduh95 aduh95 merged commit 9dfbe18 into nodejs:main Jan 26, 2025
12 checks passed
@MikeMcC399 MikeMcC399 deleted the docs/debug branch January 26, 2025 07:57
@MikeMcC399
Copy link
Contributor Author

MikeMcC399 commented Jan 26, 2025

@aduh95

Thank you for merging the documentation change! 👍🏻

When the CI test ran on the merge it has failed on random configurations with:

❯ tests/config.test.ts (1 test | 1 failed) 158ms
× key store should be up-to-date > should contain up-to-date npm keys

See https://github.com/nodejs/corepack/actions/runs/12969630436

It looks like npm is in the process of updating their keys and that some servers have been updated, and others have not. This should probably be tracked in a separate issue if it doesn't resolve itself on its own.

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.

Document DEBUG environment variable
2 participants