-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Support pipenv #888
Support pipenv #888
Conversation
Codecov Report
@@ Coverage Diff @@
## master #888 +/- ##
==========================================
+ Coverage 64.08% 64.58% +0.49%
==========================================
Files 257 259 +2
Lines 11855 11980 +125
Branches 2110 2128 +18
==========================================
+ Hits 7597 7737 +140
+ Misses 4249 4234 -15
Partials 9 9
Continue to review full report at Codecov.
|
// tslint:disable-next-line:no-empty | ||
} catch (error) { | ||
const appShell = this.serviceContainer.get<IApplicationShell>(IApplicationShell); | ||
appShell.showWarningMessage(`Workspace contains pipfile but attempt to run 'pipenv --venv' failed with $${error}. Make sure pipenv is on the PATH.`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$$
typo, will fix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MikhailArkhipov reminder.
// tslint:disable-next-line:no-empty | ||
} catch (error) { | ||
const appShell = this.serviceContainer.get<IApplicationShell>(IApplicationShell); | ||
appShell.showWarningMessage(`Workspace contains pipfile but attempt to run 'pipenv --venv' failed with $${error}. Make sure pipenv is on the PATH.`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MikhailArkhipov reminder.
Fixes #404