-
-
Notifications
You must be signed in to change notification settings - Fork 611
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
show via for direct deps #881
Comments
Or:
to avoid squatting any possible future package name namespace |
Hello @graingert! What do you mean by "direct deps"? |
Deps discovered in the .in file |
I see. Could you please provide a real example of the current behavior and what is you proposing? I want to be sure that i understand the issue. |
The current behaviour is that some dependencies in the generated requirements.txt do not include a # via comment. I'd prefer it if all dependencies included a # via comment |
Could you show an example? |
No it's not possible because the comment is missing |
Okay, let's suppose: $ cat requiremets.in
django
$ pip-compile
#
# This file is autogenerated by pip-compile
# To update, run:
#
# pip-compile
#
django==2.2.4
pytz==2019.2 # via django
sqlparse==0.3.0 # via django The $ cat requirements.txt
#
# This file is autogenerated by pip-compile
# To update, run:
#
# pip-compile
#
django==2.2.4 # via requirements.in
pytz==2019.2 # via django
sqlparse==0.3.0 # via django |
Instead I'd like to see
|
I would like to see a feature like this. It'd be nice to know the dependencies between my explicitly declared packages. |
I welcome feedback on the state of #1058. At the time of this comment: $ cat requirements.in
django
django-debug-toolbar
$ pip-compile --no-header requirements.in
asgiref==3.2.3 # via django
django-debug-toolbar==2.2 # via -r requirements.in (line 2)
django==3.0.3 # via -r requirements.in (line 1), django-debug-toolbar
pytz==2019.3 # via django
sqlparse==0.3.0 # via django, django-debug-toolbar |
…y and VCS reqs; fixes jazzband#881 and jazzband#293 Change Notes ------------ - Annotations now include reqsin/setup.py/stdin info when relevant - Always send reverse_dependencies to writer._format_requirement as a dict, even if empty - Invert checks for primary requirements not getting annotations, as they now do - Bypass annotation noise in tests which don't target annotations anyway, by passing --no-annotate - ireqs returned by get_best_match now have the parameter ireq's _source_ireqs, if any - reverse_dependencies are no longer passed around independently outside the resolver/cache, but rather determined from an ireq's comes_from and _source_ireqs - primary_packages are no longer passed around independently, but rather determined by an ireq's constraint - In tests, reverse dependencies for fake ireqs are now specified by setting the ireq's comes_from - These changes do _not_ improve handling of relative paths to reqsin files in annotations (see jazzband#1067) Co-authored-by: Albert Tugushev <albert@tugushev.ru>
Currently when compiling, direct deps do not include a 'via' declaration. eg:
however it is missing from direct dependencies.
I propose:
for direct deps
The text was updated successfully, but these errors were encountered: