resolve type aliases - #6944
Conversation
Greptile SummaryThe PR adds resolution of bare, parameterized, variadic, and union-nested type aliases before
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| packages/reflex-base/src/reflex_base/utils/types.py | Adds recursive TypeAliasType unwrapping and generic parameter substitution, including TypeVarTuple handling. |
| packages/reflex-base/src/reflex_base/vars/base.py | Resolves aliases before existing Var.guess_type dispatch. |
| tests/units/reflex_base/vars/test_base.py | Places alias regression tests at the source-corresponding path requested by the previous review thread and covers bare, generic, variadic, optional, and state-field aliases. |
| packages/reflex-base/news/6944.bugfix.md | Documents the newly supported type-alias forms and prior compilation failure. |
Reviews (6): 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.