docs: import Literal in the metadata-expansion example#184
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe README metadata example import now includes ChangesREADME example update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
Hey, just wanted to drop a quick note as Q2 closes out — wanted to make sure this didn't get lost in the queue. This is a one-line docs fix: the Let me know if there's anything blocking on your end! |
The "expand the document metadata" example in the README uses
Literal[...]for thetopicsfield but only importsAnnotatedfromtyping. Copy-pasting the snippet as-is raisesNameError: name Literal is not defined.This one-line fix adds
Literalto the import so the example runs as written. Docs-only — no logic change.Reproduction (before fix):
Summary by CodeRabbit