Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Bug: nx 20 breaks yarn knip #10174

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

Open
4 tasks done
abrahamguo opened this issue Oct 18, 2024 · 3 comments
Open
4 tasks done

Bug: nx 20 breaks yarn knip #10174

abrahamguo opened this issue Oct 18, 2024 · 3 comments
Assignees
Labels
accepting prs Go ahead, send a pull request that resolves this issue bug Something isn't working repo maintenance things to do with maintenance of the repo, and not with code/docs

Comments

@abrahamguo
Copy link
Contributor

abrahamguo commented Oct 18, 2024

Before You File a Bug Report Please Confirm You Have Done The Following...

  • I have tried restarting my IDE and the issue persists.
  • I have updated to the latest version of the packages.
  • I have searched for related issues and found none that matched my issue.
  • I have read the FAQ and my problem is not listed.

Issue Description

Since #10128, yarn knip fails locally on my macOS with

 NX   Daemon process terminated and closed the connection

Please rerun the command, which will restart the daemon.
If you get this error again, check for any errors in the daemon process logs found in: typescript-eslint/.nx/workspace-data/d/daemon.log

The log file has nothing in it that appears to be a failure.

Reproduction Repository Link

https://github.com/typescript-eslint/typescript-eslint

Repro Steps

  1. clone the repo
  2. yarn
  3. yarn knip

Versions

package version
node 22.9.0
yarn 3.8.2
@abrahamguo abrahamguo added bug Something isn't working triage Waiting for team members to take a look labels Oct 18, 2024
@auvred
Copy link
Member

auvred commented Oct 21, 2024

Same thing! It looks like NX is being called from jest.config.js.

@abrahamguo
Copy link
Contributor Author

abrahamguo commented Oct 21, 2024

FYI not a permanent solution, but I was able to work around it using NX_DAEMON=false from knip/727.

@JoshuaKGoldberg
Copy link
Member

Huh, I get this too. cc @JamesHenry - I've never seen it before with Knip and am confused.

@JoshuaKGoldberg JoshuaKGoldberg added repo maintenance things to do with maintenance of the repo, and not with code/docs accepting prs Go ahead, send a pull request that resolves this issue and removed triage Waiting for team members to take a look labels Dec 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepting prs Go ahead, send a pull request that resolves this issue bug Something isn't working repo maintenance things to do with maintenance of the repo, and not with code/docs
Projects
None yet
Development

No branches or pull requests

4 participants