Skip to content

Bump setuptools minimum version to 78.1.1 - #126

Closed
t-uda wants to merge 1 commit into
ellphi-0.1.3from
claude/init-session-DHa05
Closed

Bump setuptools minimum version to 78.1.1#126
t-uda wants to merge 1 commit into
ellphi-0.1.3from
claude/init-session-DHa05

Conversation

@t-uda

@t-uda t-uda commented May 16, 2026

Copy link
Copy Markdown
Owner

Summary

Updates the minimum required version of setuptools in the build system dependencies from 65 to 78.1.1.

Changes

  • Updated pyproject.toml build-system requirements to specify setuptools>=78.1.1 instead of setuptools>=65

Details

This change ensures the project uses a more recent version of setuptools that likely includes important bug fixes, security updates, or features needed for the build process. The version bump from 65 to 78.1.1 represents a significant update that may address compatibility issues or improve build reliability.

https://claude.ai/code/session_01CUuVVqiQr32khYnEBz1eKD

…25-49

- Resolves path traversal vulnerability in setuptools (PYSEC-2025-49)
- Also addresses related CVE-2026-24049 in wheel
- Required for secure package building and installation

https://claude.ai/code/session_01CUuVVqiQr32khYnEBz1eKD
Copilot AI review requested due to automatic review settings May 16, 2026 07:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Bumps the minimum required setuptools version in the build-system requirements from 65 to 78.1.1, likely to pull in security fixes (notably CVE-2025-47273 affecting setuptools < 78.1.1).

Changes:

  • Update setuptools minimum version constraint in pyproject.toml build-system requires.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov

codecov Bot commented May 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@t-uda
t-uda changed the base branch from main to ellphi-0.1.3 July 3, 2026 12:22
t-uda added a commit that referenced this pull request Jul 3, 2026
* chore: migrate dependency management from Poetry to uv

- Move dev/docs tooling to PEP 735 [dependency-groups]; drop demo
  extras and all Poetry groups (demo lives in the ellphi-demo repo)
- Replace poetry.lock with uv.lock; constrain numpy<2.5 in the dev
  lockfile until stubtest supports numpy's PEP 695 NDArray alias
- Keep the poetry-core build backend so pip/uv installs still compile
  the C++ tangency backend; bump setuptools floor to >=78.1.1
  (CVE-2025-47273, subsumes PR #126) and poetry-core to >=2.0
- Switch all four workflows to astral-sh/setup-uv with caching,
  uv sync/run/build; add a `uv lock --check` freshness step
- Repurpose the demo-install job to smoke-test a plain sdist install
- Bump codecov-action to v5 and make the upload non-blocking (#117)
- Update README, CONTRIBUTING, AGENTS and docs to uv commands
- Drop the ellphi.solver.MethodName stubtest allowlist entry, unused
  since mypy 1.20

Closes #121
Closes #117

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* chore: address review feedback on uv migration

- scripts/hybrid_tuning.sh: migrate `poetry run` to `uv run`
- docs/eigen_backend_plan.md: include ELLPHI_USE_EIGEN=1 in the
  rebuild command so the extension is rebuilt in Eigen mode
- src/ellphi/_tangency_cpp.py: rebuild hint now covers both uv and
  pip (`pip install --force-reinstall .`) and uses the correct
  scripts/build_tangency_cpp.py path
- .github/workflows/python-app.yml: fix "explicitely" typo
- README.md: clarify that maintained demo notebooks live in
  ellphi-demo while in-repo notebooks/ examples remain but their
  extra dependencies are no longer declared by this package

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Tomoki Uda <tomoki.uda+amenosagiri@gmail.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
@t-uda

t-uda commented Jul 3, 2026

Copy link
Copy Markdown
Owner Author

Closing as subsumed by PR #132 (merged): the migrated pyproject.toml carries the setuptools>=78.1.1 floor in [build-system] (CVE-2025-47273 defense-in-depth), so this one-liner is already on ellphi-0.1.3. Thanks — the floor bump survived the migration as intended.

@t-uda t-uda closed this Jul 3, 2026
@t-uda
t-uda deleted the claude/init-session-DHa05 branch July 3, 2026 13:03
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.

3 participants