fedora-python / fedora-python/tox-current-env

print-deps-only: Document we don't handle recursive deps (from files, e.g. -rrequirements.txt)

Open
#22 8 comments 0 reactions 0 assignees View on GitHub
documentation good first issue
Dominant language
Python
Stars
26
Forks
9
PR merge metrics
No merged PRs in 30d

Description

Fedora report: https://bugzilla.redhat.com/show_bug.cgi?id=1808601

tox docs: https://tox.readthedocs.io/en/latest/example/basic.html#depending-on-requirements-txt-or-defining-constraints

When the dependencies are defined as:

```
[testenv]
deps = -rrequirements.txt
```

We need to read the files (recursively) and output their content.

The code that handles this now is https://github.com/fedora-python/tox-current-env/blob/dc47ce3ddf53c2b950f0a776ea6ebcc7355e3522/src/tox_current_env/hooks.py#L161-L172

The `*venv.get_resolved_dependencies()` call needs to be replaced with a custom function call. That function needs to call `venv.get_resolved_dependencies()` and handle "packages" that start with `-r`/`-c` specially.

- https://pip.pypa.io/en/stable/user_guide/#requirements-files
- https://pip.pypa.io/en/stable/user_guide/#constraints-files

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.