Skip to content

Fix pip-install import test shadowed by local source#339

Merged
RoryBarnes merged 1 commit intomainfrom
fix-pip-install
Feb 19, 2026
Merged

Fix pip-install import test shadowed by local source#339
RoryBarnes merged 1 commit intomainfrom
fix-pip-install

Conversation

@RoryBarnes
Copy link
Copy Markdown
Collaborator

Summary

  • The sdist/wheel import verification (import vplanet) ran from the repo root, where the local vplanet/ source directory shadows the installed package in the venv
  • The local source lacks the compiled C extension (vplanet_core), causing ModuleNotFoundError
  • Fix: cd /tmp before running the import so Python finds the installed package, not the local source

Test plan

  • Merge and publish a new release to trigger pip-install workflow
  • Verify "Test the sdist" and "Test the wheel" steps pass

🤖 Generated with Claude Code

The import verification runs from the repo root, where the local vplanet/
source directory shadows the installed package. The local source lacks the
compiled C extension (vplanet_core), causing ModuleNotFoundError. Fix by
cd'ing to /tmp before running the import test.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@github-actions
Copy link
Copy Markdown

Test Results (py3.13 on ubuntu-24.04)

19 903 tests   19 647 ✅  1m 34s ⏱️
     1 suites       0 💤
     1 files       256 ❌

For more details on these failures, see this check.

Results for commit 9eb66e2.

@github-actions
Copy link
Copy Markdown

Test Results (py3.12 on ubuntu-24.04)

19 903 tests   19 647 ✅  1m 36s ⏱️
     1 suites       0 💤
     1 files       256 ❌

For more details on these failures, see this check.

Results for commit 9eb66e2.

@github-actions
Copy link
Copy Markdown

Test Results (py3.11 on ubuntu-22.04)

19 903 tests   19 647 ✅  1m 35s ⏱️
     1 suites       0 💤
     1 files       256 ❌

For more details on these failures, see this check.

Results for commit 9eb66e2.

@github-actions
Copy link
Copy Markdown

Test Results (py3.11 on ubuntu-24.04)

19 903 tests   19 647 ✅  1m 34s ⏱️
     1 suites       0 💤
     1 files       256 ❌

For more details on these failures, see this check.

Results for commit 9eb66e2.

@github-actions
Copy link
Copy Markdown

Test Results (py3.10 on ubuntu-22.04)

19 903 tests   19 647 ✅  1m 40s ⏱️
     1 suites       0 💤
     1 files       256 ❌

For more details on these failures, see this check.

Results for commit 9eb66e2.

@github-actions
Copy link
Copy Markdown

Test Results (py3.14 on ubuntu-22.04)

19 903 tests   19 647 ✅  1m 37s ⏱️
     1 suites       0 💤
     1 files       256 ❌

For more details on these failures, see this check.

Results for commit 9eb66e2.

@github-actions
Copy link
Copy Markdown

Test Results (py3.10 on ubuntu-24.04)

19 903 tests   19 647 ✅  1m 41s ⏱️
     1 suites       0 💤
     1 files       256 ❌

For more details on these failures, see this check.

Results for commit 9eb66e2.

@github-actions
Copy link
Copy Markdown

Test Results (py3.14 on ubuntu-24.04)

19 903 tests   19 647 ✅  1m 34s ⏱️
     1 suites       0 💤
     1 files       256 ❌

For more details on these failures, see this check.

Results for commit 9eb66e2.

@github-actions
Copy link
Copy Markdown

Test Results (py3.9 on ubuntu-22.04)

19 903 tests   19 647 ✅  1m 43s ⏱️
     1 suites       0 💤
     1 files       256 ❌

For more details on these failures, see this check.

Results for commit 9eb66e2.

@github-actions
Copy link
Copy Markdown

Test Results (py3.13 on ubuntu-22.04)

19 903 tests   19 647 ✅  1m 37s ⏱️
     1 suites       0 💤
     1 files       256 ❌

For more details on these failures, see this check.

Results for commit 9eb66e2.

@github-actions
Copy link
Copy Markdown

Test Results (py3.12 on ubuntu-22.04)

19 903 tests   19 647 ✅  1m 38s ⏱️
     1 suites       0 💤
     1 files       256 ❌

For more details on these failures, see this check.

Results for commit 9eb66e2.

@github-actions
Copy link
Copy Markdown

Test Results (py3.9 on ubuntu-24.04)

19 903 tests   19 647 ✅  1m 43s ⏱️
     1 suites       0 💤
     1 files       256 ❌

For more details on these failures, see this check.

Results for commit 9eb66e2.

@RoryBarnes RoryBarnes merged commit e3f6759 into main Feb 19, 2026
1 of 38 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