Skip to content

Conversation

@bbopen
Copy link
Owner

@bbopen bbopen commented Jan 20, 2026

Summary

The PyPI publish workflow was failing because the Python API test was finding tywrap_ir as a namespace package from the repo root instead of the installed package.

Fix

Run the test from /tmp directory to avoid the namespace conflict.

Test

  • PyPI publish workflow succeeds

Generated with Claude Code

The test was finding tywrap_ir as a namespace package from the repo
root instead of the installed package.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@coderabbitai
Copy link

coderabbitai bot commented Jan 20, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The pull request modifies a GitHub Actions workflow by adding a working-directory: /tmp parameter to the "Test Python API" step in the publish-pypi workflow, directing the Python test execution to run from the /tmp directory instead of the default working directory.

Changes

Cohort / File(s) Summary
GitHub Actions Configuration
.github/workflows/publish-pypi.yml
Added working-directory: /tmp to the "Test Python API" step to specify the execution directory for Python imports and script execution.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A hop through workflows, a path set so clear,
To /tmp we venture, no need for fear,
One line of magic, so simple, so sweet,
The test runs faster in this new seat! ✨


📜 Recent review details

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b045fc8 and f0de936.

📒 Files selected for processing (1)
  • .github/workflows/publish-pypi.yml

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


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

@bbopen bbopen merged commit 4994f06 into main Jan 20, 2026
18 of 19 checks passed
@bbopen bbopen deleted the fix/pypi-namespace-conflict branch January 20, 2026 00:02
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.

2 participants