Skip to content

python3Packages.smolagents: split pytest checks to passthru.tests#469544

Draft
ShamrockLee wants to merge 1 commit intoNixOS:masterfrom
ShamrockLee:smolagents-split-tests
Draft

python3Packages.smolagents: split pytest checks to passthru.tests#469544
ShamrockLee wants to merge 1 commit intoNixOS:masterfrom
ShamrockLee:smolagents-split-tests

Conversation

@ShamrockLee
Copy link
Contributor

Reduce closure size by splitting tests, as the tests depends on optional-dependencies.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

Reduce closure size by splitting tests,
as the tests depends on optional-dependencies.
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 6.topic: python Python is a high-level, general-purpose programming language. labels Dec 10, 2025
@ShamrockLee
Copy link
Contributor Author

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 469544 -a python312Packages.smolagents.tests -a python313Packages.smolagents.tests
Commit: b130c102f291d712ccd4831b58b02fadc508eb35 (subsequent changes)
Merge: 46388f7e7d03b31f08b62cef2b953e9ac341d8ba

Logs: https://github.com/ShamrockLee/nixpkgs-review-gha/actions/runs/20113738775


x86_64-linux

❌ 4 packages failed to build:
  • python312Packages.smolagents.tests.pytest-check
  • python312Packages.smolagents.tests.pytest-check.dist
  • python312Packages.txtai
  • python312Packages.txtai.dist
✅ 8 packages built:
  • python312Packages.smolagents
  • python312Packages.smolagents.dist
  • python313Packages.smolagents
  • python313Packages.smolagents.dist
  • python313Packages.smolagents.tests.pytest-check
  • python313Packages.smolagents.tests.pytest-check.dist
  • python313Packages.txtai
  • python313Packages.txtai.dist

aarch64-linux

✅ 12 packages built:
  • python312Packages.smolagents
  • python312Packages.smolagents.dist
  • python312Packages.smolagents.tests.pytest-check
  • python312Packages.smolagents.tests.pytest-check.dist
  • python312Packages.txtai
  • python312Packages.txtai.dist
  • python313Packages.smolagents
  • python313Packages.smolagents.dist
  • python313Packages.smolagents.tests.pytest-check
  • python313Packages.smolagents.tests.pytest-check.dist
  • python313Packages.txtai
  • python313Packages.txtai.dist

x86_64-darwin (sandbox = relaxed)

❌ 4 packages failed to build:
  • python312Packages.smolagents.tests.pytest-check
  • python312Packages.smolagents.tests.pytest-check.dist
  • python313Packages.smolagents.tests.pytest-check
  • python313Packages.smolagents.tests.pytest-check.dist
✅ 4 packages built:
  • python312Packages.smolagents
  • python312Packages.smolagents.dist
  • python313Packages.smolagents
  • python313Packages.smolagents.dist

aarch64-darwin (sandbox = relaxed)

❌ 4 packages failed to build:
  • python312Packages.smolagents.tests.pytest-check
  • python312Packages.smolagents.tests.pytest-check.dist
  • python313Packages.smolagents.tests.pytest-check
  • python313Packages.smolagents.tests.pytest-check.dist
✅ 4 packages built:
  • python312Packages.smolagents
  • python312Packages.smolagents.dist
  • python313Packages.smolagents
  • python313Packages.smolagents.dist

@nixpkgs-ci nixpkgs-ci bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Jan 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.status: merge conflict This PR has merge conflicts with the target branch 6.topic: python Python is a high-level, general-purpose programming language. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant