Skip to content

Commit 6594342

Browse files
sspickleclaude
andcommitted
fix: the del-cleanup lines still referenced the renamed imports
The pkg_resources -> importlib.metadata change renamed the imports but missed the cleanup three lines below, which still did `del get_distribution` — so `import vpython` raised NameError on every platform, and the whole matrix went red on the previous push. Caught by that CI run. The local check that let it through was a syntax parse, which `del wrong_name` passes; the block is now verified by executing it, which is what should have happened the first time. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01G7y9rTA1r8r8EhEnPSQenR
1 parent 31ed45b commit 6594342

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

vpython/__init__.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
__gs_version__ = glowscript_version()
1414

1515
del glowscript_version
16-
del get_distribution
17-
del DistributionNotFound
16+
del _dist_version
17+
del PackageNotFoundError
1818

1919
# Keep the remaining imports later to ensure that __version__ and
2020
# __gs_version__ exist before importing vpython, which itself imports

0 commit comments

Comments
 (0)