resolve type aliases - #6944
Conversation
Greptile SummaryThe PR teaches
Confidence Score: 5/5The PR appears safe to merge because the previous test-location issue is fixed and no blocking failure remains. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| packages/reflex-base/src/reflex_base/utils/types.py | Adds helpers for resolving bare, parameterized, variadic, and union-nested type aliases. |
| packages/reflex-base/src/reflex_base/vars/base.py | Resolves aliases before the existing Var.guess_type dispatch. |
| tests/units/reflex_base/vars/test_base.py | Adds regression tests in the source-corresponding reflex-base test location, resolving the prior review finding. |
| packages/reflex-base/news/6944.bugfix.md | Documents support for resolving type-alias annotations during variable type inference. |
Reviews (7): Last reviewed commit: "Merge branch 'main' into allow-typevar-r..." | Re-trigger Greptile
Merging this PR will not alter performance
Comparing Footnotes
|
572b04c to
fb3c49e
Compare
There was a problem hiding this comment.
All reported issues were addressed across 4 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 3 files (changes from recent commits).
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
Uh oh!
There was an error while loading. Please reload this page.