Skip to content

Updating Workflows to Fix Missing pkg_resources#2057

Open
ericspod wants to merge 3 commits intoProject-MONAI:mainfrom
ericspod:pkg_resources_fix
Open

Updating Workflows to Fix Missing pkg_resources#2057
ericspod wants to merge 3 commits intoProject-MONAI:mainfrom
ericspod:pkg_resources_fix

Conversation

@ericspod
Copy link
Member

@ericspod ericspod commented Feb 13, 2026

Description

This updates the workflow files to fix changes to setuptools which drops support for pkg_resources but also removes the package from PyPI. With the --no-build-isolation flag added to pip, the version of setuptools in the installing environment is used, rather than a new one being installed in an isolated one which will be too new and fail when pkg_resources is found missing.

See Project-MONAI/MONAI#8536.

Checks

  • Avoid including large-size files in the PR.
  • Clean up long text outputs from code cells in the notebook.
  • For security purposes, please check the contents and remove any sensitive info such as user names and private key.
  • Ensure (1) hyperlinks and markdown anchors are working (2) use relative paths for tutorial repo files (3) put figure and graphs in the ./figure folder
  • Notebook runs automatically ./runner.sh -t <path to .ipynb file>

Summary by CodeRabbit

  • Chores
    • Updated CI workflows to newer action versions and upgraded Python runtime to 3.10
    • Simplified and consolidated dependency caching into a unified pip cache, removing legacy timestamp caching
    • Streamlined dependency installation and test/lint invocations (including upgraded packaging tooling and adjustments to build isolation) for more consistent and reliable CI runs

Signed-off-by: Eric Kerfoot <17726042+ericspod@users.noreply.github.com>
@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

Walkthrough

Multiple GitHub Actions workflows updated: actions/checkout and actions/setup-python bumped to v6, Python target bumped to 3.10, caching simplified to cache: 'pip', and dependency installation lines adjusted (added wheel-stub, removed remote dev requirements, some --no-build-isolation usage and minor command form changes).

Changes

Cohort / File(s) Summary
CI workflows
.github/workflows/copyright.yml, .github/workflows/guidelines.yml, .github/workflows/pep8.yml, .github/workflows/test-modified.yml
Updated actions/checkout and actions/setup-python to v6 and Python to 3.10; replaced legacy timestamp/multi-step caching with cache: 'pip'; consolidated dependency install steps to use pip, added wheel-stub, removed remote requirements-dev.txt installs in places, and adjusted --no-build-isolation usage and minor command forms (e.g., python -m pippip).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰🥕 I hopped through YAML, tidy and bright,
Bumped versions, trimmed caches, set Python right,
Wheel-stub tucked in, pip dances in line,
CI carrots gleam — short, clean, and fine,
A little rabbit cheers the green light.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main purpose of the changes: updating workflows to resolve the missing pkg_resources issue caused by setuptools changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main
Description check ✅ Passed The pull request description includes a clear explanation of the changes and references the upstream issue, but uses the unchecked checklist format rather than explicitly confirming checks.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]

This comment was marked as outdated.

Signed-off-by: Eric Kerfoot <17726042+ericspod@users.noreply.github.com>
Signed-off-by: Eric Kerfoot <17726042+ericspod@users.noreply.github.com>
@ericspod ericspod requested review from KumoLiu and Nic-Ma February 13, 2026 19:18
@ericspod ericspod enabled auto-merge (squash) February 13, 2026 19:20
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