-
Notifications
You must be signed in to change notification settings - Fork 31
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
Added sublime_lib dependency. #157
Conversation
@randy3k Any chance you can take a look at the PowerShell stuff? I've never worked with it and I don't have a Windows machine to test it with. |
@@ -68,6 +68,24 @@ Bootstrap() { | |||
echo | |||
fi | |||
|
|||
SUBLIME_LIB_PATH="$STP/coverage" |
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.
This path should $STP/sublime_lib
@@ -73,6 +73,24 @@ Bootstrap() { | |||
echo | |||
fi | |||
|
|||
SUBLIME_LIB_PATH="$STP/coverage" |
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.
same here
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.
Sorry for the slow reply. It may take a bit longer time to update powershell scripts (as those scripts are not written by me).
7690bbf
to
fbf0a56
Compare
If a package requests a library, it is installed by Package Control. |
Will be used for #155, #67, and probably many other things.
I'm not familiar with PowerShell; we'll need to add sublime_lib in there as well. I also have not tested the CI integration.
Ideally, we could move most of the installation code into Python, relying on
pathlib
and other built-in cross-platform functionality. In a perfect world, it would be nice to make this a full-fledged pypi package.