Add inline dependencies for project fixtures#47
Conversation
Project fixtures now scan PEP 723 script metadata before import and install declared packages through the active Python environment. This shares the existing Python-test behavior with normal fixture loading and standalone fixture mode. Assisted-by: GPT-5 (Codex)
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6cdc1172df
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 562d64f5f7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
mavam
force-pushed
the
topic/pep723-fixtures
branch
from
562d64f to
344c9a4
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 344c9a43a3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Match inline dependency discovery to the fixture module import order. Package fixtures now ignore stale top-level files and nested modules that the loader would not import directly. Assisted-by: ChatGPT (pi)
mavam
force-pushed
the
topic/pep723-fixtures
branch
from
344c9a4 to
9d9a8a4
Compare
1 participant
🔍 Problem
installed PEP 723 dependencies for Python test scripts.
uvx --with ..., andstandalone fixture mode had the same gap.
🛠️ Solution
project fixture loading.
fixtures.pyand recursivefixtures/**/*.pyfiles before importingproject fixtures.
order, recursive scans, empty metadata, invalid metadata, and undeclared
dependency fallbacks.
💬 Review
- Focus on the eager fixture dependency scan and whether installing all declared
📚 Docs PR: tenzir/docs#317fixture dependencies before fixture import is the right trade-off for the
current eager registration model.