Skip to content

Add missing multidict dependency#3

Merged
MiLk merged 1 commit intomainfrom
fix/add-multidict-dependency
Apr 2, 2026
Merged

Add missing multidict dependency#3
MiLk merged 1 commit intomainfrom
fix/add-multidict-dependency

Conversation

@LysanderGG
Copy link
Copy Markdown

Summary

  • twirp/exceptions.py imports multidict.CIMultiDictProxy but multidict was not declared in [project.dependencies]
  • This causes ModuleNotFoundError: No module named 'multidict' when the package is installed in a clean environment

Test plan

  • Fresh venv install: pip install -e .from twirp.exceptions import TwirpServerException succeeds

🤖 Generated with Claude Code

`twirp/exceptions.py` imports `multidict.CIMultiDictProxy` but
`multidict` was not declared in project dependencies, causing
`ModuleNotFoundError` at import time.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@LysanderGG LysanderGG requested a review from MiLk April 2, 2026 05:13
@MiLk MiLk merged commit aa17f7a into main Apr 2, 2026
4 checks passed
@MiLk MiLk deleted the fix/add-multidict-dependency branch April 2, 2026 12:18
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