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
pythonPackages.ptvsd: init at 4.3.2 #88005
Conversation
325bd42
to
259ce3b
Compare
@ptrhlm done, sorry
…On Sun, May 17, 2020 at 4:42 PM ptrhlm ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In pkgs/development/python-modules/ptvsd/default.nix
<#88005 (comment)>:
> + version = "4.3.2";
+
+ format = "wheel";
+
+ src = fetchPypi {
+ inherit pname version format;
+ sha256 = "459137736068bb02515040b2ed2738169cb30d69a38e0fd5dffcba255f41e68d";
+ };
+
+ checkInputs = [ pytest ];
+
+ meta = with stdenv.lib; {
+ homepage = "https://github.com/Microsoft/ptvsd/";
+ description = "An implementation of the Debug Adapter Protocol for Python";
+ license = licenses.mit;
+ maintainers = with maintainers; [ leungbk ];
Please add yourself to maintainers list:
https://github.com/NixOS/nixpkgs/blob/master/maintainers/maintainer-list.nix
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#88005 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AG65GOX4JI3GIRXPN7OIHO3RSBY7NANCNFSM4NDF5BVQ>
.
|
thanks for opening your first PR! :) |
Tests for `ptvsd` do not seem to be included in the `.zip`. Should we go
ahead with the `.zip` with this in mind?
edit: nvm, the wheel doesn't have tests either; I've now pushed the zip.
…On Mon, May 18, 2020 at 6:14 PM Jon ***@***.***> wrote:
thanks for opening your first PR! :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#88005 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AG65GOW2G4Y4DPZFOZ3QELDRSHMOFANCNFSM4NDF5BVQ>
.
|
e8d1a20
to
579e3e9
Compare
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.
I've tested your changes with Python 3.7 and VS Code. Seems to work. Great job
# no tests in the wheel or the zip | ||
doCheck = false; |
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.
There are no tests included. Please try to checkout from source and check if they have unit tests, and try to run them. Unit tests give a good indication that they package has a high degree of validity and correctness given the python package set.
If tests are not available, then please use pythonImportsCheck
to import the most important modules. This isn't as good as unit tests, but will usually give a good indication of run-time errors.
Apparently So I'll close this. |
Motivation for this change
Add
ptvsd
, a Python debugging tool compliant with the Debug Adapter Protocol.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)