Paweł Stradomski

Looks reasonable. These files are not defined in PEP, but it looks like setuptools includes everything that matches LICEN[CS]E, AUTHORS, NOTICE or COPYING. Should be simple to allow user to...

Should be fixed by #831

(3) makes the output filename non-compliant with the PEP, so I'd avoid it. I don't mind it that much for the cases where the stamping is used, since it's already...

There defnitely are rules that consume py_wheel output (as "data" dependencies). Can you elaborate on your use case? The current behaviour is standard-compliant in absence of stamping, and in particular...

It should be relatively straightforward to have a tarball output that contains the wheel with the correctly name file inside - and it would avoid the issues with generated directories...

> > @duggelz @AlecBenzer > it looks like this is possible using > Python toolchains, see > https://gist.github.com/NathanHowell/5cf4a353a8dd3a1025e682c4707d5bac for > an example > This is pretty awesome, I've been using...

Any background for why this is needed? Maybe the documentation should be updated to reflect that and what's the interaction with "requires" attribute?

@brandjon: it's something I have in my fork of rules_python, using a genrule to create a snippet similar to the one in @groodt comment above. Do you think it makes...

> i added a sample macro for pytest here https://github.com/ali5h/rules_pip/blob/master/defs.bzl#L116 PTAL at pull request #243. It takes a slightly different approach. In particular, it does not pull dependency on pytest...

You should be able "bazel run" a py_test target; no need to change it to py_binary first. śr., 31 mar 2021 o 01:23 Jonathon Belotti ***@***.***> napisał(a): > @jxramos RE...