Skip to content

Better default value for jest-executable #21

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
davazp opened this issue Mar 6, 2021 · 0 comments
Open

Better default value for jest-executable #21

davazp opened this issue Mar 6, 2021 · 0 comments

Comments

@davazp
Copy link

davazp commented Mar 6, 2021

I have been setting up jest.el for my projects and noticed that the default value of jest-executable (npm test) is not that general.

In my case, the default breaks because we work in a monorepository where the toplevel test is not directly invoking jest.

Proposed solution

I would propose to change it to npx --no-install jest, which should work in a larger number of cases.

The downside is you need a bit more modern version of npm. But it was introduced a while back.

Alternative solution

Alternatively, finding "jest" in the path would also work and the user could rely on add-node-modules-path. Related discussion: #2. But my proposed default value should work in most installations.

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

1 participant