Skip to content

[codex] Fix Poetry extras dependency declarations#1842

Merged
FlorianPfaff merged 2 commits intomainfrom
codex/fix-poetry-extras
Apr 25, 2026
Merged

[codex] Fix Poetry extras dependency declarations#1842
FlorianPfaff merged 2 commits intomainfrom
codex/fix-poetry-extras

Conversation

@FlorianPfaff
Copy link
Copy Markdown
Owner

Summary

  • Declare the optional backend packages in the main Poetry dependency table with optional = true.
  • Keep the same packages in the dev dependency group so the existing development/test install behavior still includes all backends.
  • Leave the existing extras groups wired to those optional dependencies so package installs like pyrecest[jax_support], pyrecest[pytorch_support], and pyrecest[healpy_support] can emit the expected runtime dependency metadata.

Root cause

The extras listed healpy, torch, jax, jaxlib, and autograd, but those packages were only declared under [tool.poetry.group.dev.dependencies]. Dev dependency groups are not package runtime metadata, so the published extras could resolve to empty dependency sets for users installing from PyPI.

Validation

  • Verified the branch diff contains only the intended pyproject.toml change.
  • Not run locally; this Codex session has a read-only filesystem and cannot regenerate or validate poetry.lock/exported requirements. Before marking ready, run poetry lock and refresh generated requirement exports if the project requires them in the same PR.

@FlorianPfaff FlorianPfaff marked this pull request as ready for review April 25, 2026 07:45
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 25, 2026

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ COPYPASTE jscpd yes no no 12.42s
✅ JSON prettier 2 0 0 0 0.45s
✅ JSON v8r 2 0 0 2.49s
✅ MARKDOWN markdownlint 2 0 0 0 0.69s
✅ MARKDOWN markdown-table-formatter 2 0 0 0 0.26s
✅ PYTHON bandit 382 0 0 5.39s
✅ PYTHON black 382 33 0 0 9.46s
✅ PYTHON flake8 382 0 0 2.92s
✅ PYTHON isort 382 36 0 0 0.73s
✅ PYTHON mypy 382 0 0 4.82s
✅ PYTHON pylint 382 0 0 113.97s
✅ PYTHON ruff 382 36 0 0 0.08s
✅ REPOSITORY checkov yes no no 22.35s
✅ REPOSITORY gitleaks yes no no 6.76s
✅ REPOSITORY git_diff yes no no 0.03s
✅ REPOSITORY secretlint yes no no 7.33s
✅ REPOSITORY syft yes no no 6.53s
✅ REPOSITORY trivy-sbom yes no no 2.96s
✅ REPOSITORY trufflehog yes no no 18.6s
✅ YAML prettier 4 0 0 0 0.5s
✅ YAML v8r 4 0 0 4.55s
✅ YAML yamllint 4 0 0 0.43s

See detailed reports in MegaLinter artifacts

Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining FLAVOR_SUGGESTIONS: false)

  • Documentation: Custom Flavors
  • Command: npx mega-linter-runner@9.4.0 --custom-flavor-setup --custom-flavor-linters PYTHON_PYLINT,PYTHON_BLACK,PYTHON_FLAKE8,PYTHON_ISORT,PYTHON_BANDIT,PYTHON_MYPY,PYTHON_RUFF,COPYPASTE_JSCPD,JSON_V8R,JSON_PRETTIER,MARKDOWN_MARKDOWNLINT,MARKDOWN_MARKDOWN_TABLE_FORMATTER,REPOSITORY_CHECKOV,REPOSITORY_GIT_DIFF,REPOSITORY_GITLEAKS,REPOSITORY_SECRETLINT,REPOSITORY_SYFT,REPOSITORY_TRIVY_SBOM,REPOSITORY_TRUFFLEHOG,YAML_PRETTIER,YAML_YAMLLINT,YAML_V8R

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 25, 2026

Test Results

    12 files  +    4      12 suites  +4   4h 39m 36s ⏱️ + 2h 53m 56s
 1 106 tests +   16   1 106 ✅ +   16      0 💤 ±  0  0 ❌ ±0 
13 556 runs  +4 624  10 408 ✅ +3 804  3 148 💤 +820  0 ❌ ±0 

Results for commit 1c145f6. ± Comparison against base commit 24543a9.

♻️ This comment has been updated with latest results.

@FlorianPfaff FlorianPfaff merged commit 875a98e into main Apr 25, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant