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

auto recovery with login-shell when encountered command not found issue #941

Merged
merged 4 commits into from
Nov 11, 2022

Conversation

connectdotz
Copy link
Collaborator

There are many complaints about intermittent test run failure (exit code 127) with "command not found" or "No such file or directory" errors for programs like node/npm/yarn. While it is not caused by this extension, it significantly impacted our users' experience nevertheless.

With the recent implementation of the login-shell, I think we can finally address this once and for all. This PR implemented an auto-retry with a login shell when detected the situation mentioned above.

This is only applicable for non-windows platforms. The currently supported login shell are: bash, zsh, fish.

fixes #899
fixes #845
fixes #874
fixes #867

@coveralls
Copy link

coveralls commented Nov 11, 2022

Pull Request Test Coverage Report for Build 3442324402

  • 75 of 75 (100.0%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.04%) to 96.29%

Totals Coverage Status
Change from base Build 3441975667: 0.04%
Covered Lines: 3131
Relevant Lines: 3195

💛 - Coveralls

@connectdotz connectdotz changed the title auto fallback to login-shell when encountered command not found issue auto recovery with login-shell when encountered command not found issue Nov 11, 2022
@connectdotz connectdotz merged commit 9c6f121 into jest-community:master Nov 11, 2022
@connectdotz connectdotz deleted the fall-back-login-shell branch November 11, 2022 22:52
legend1202 pushed a commit to legend1202/vscode-jest that referenced this pull request Jun 18, 2023
…ue (jest-community#941)

* added fallback to login-shell logic

* update doc

* adding test and correct a regex
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants