Conversation
masenf
left a comment
There was a problem hiding this comment.
update state_auto_setters in BaseConfig
Greptile SummaryThis PR removes or delays a batch of 0.9.0 deprecations across the Reflex codebase. Non-string key support in
Confidence Score: 5/5Safe to merge — all remaining findings are minor documentation issues with no runtime impact. Both findings are P2 stale-docstring issues introduced by the deprecation removal commits. There are no logic errors, data-integrity concerns, or behavioral regressions. The delayed deprecations correctly preserve backward compatibility for users still on those paths. reflex/state.py and reflex/app.py have minor stale docstrings that should be cleaned up, but neither blocks merging. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[PR: Remove 090 Deprecations] --> B{Action}
B -->|Remove Now| C[get_value non-string keys\nreflex/state.py]
B -->|Remove Now| D[overlay_component field\nreflex/app.py]
B -->|Remove Now| E[codeblock deprecation warnings\nmarkdown.py]
B -->|Remove Now| F[state_auto_setters warnings\nconfig.py]
B -->|Delay to v1.0| G[RouterData.page property\nistate/data.py]
B -->|Delay to v1.0| H[disable_plugins string support\nconfig.py]
C --> I[⚠️ Stale docstring: 'or MutableProxy']
D --> J[⚠️ Stale docstring: overlay_component attribute]
G --> K[deprecation_version=0.8.1\nremoval_version=1.0]
H --> L[deprecation_version=0.8.28\nremoval_version=1.0]
|
No description provided.