-
Notifications
You must be signed in to change notification settings - Fork 1.7k
0817dev #5916
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
0817dev #5916
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Greptile Overview
Summary
Development version bump from 0.8.16dev1 to 0.8.17dev1 with routine dependency updates across Python and JavaScript libraries.
Key Changes
- Version: Bumped to 0.8.17dev1
- Python Dependencies: Updated
reflex-hosting-cli(0.1.55→0.1.57),wraptupper bound (<2.0→<3.0), andruffpre-commit hook (v0.14.0→v0.14.1) - Frontend Libraries: Updated lucide-react (0.545.0→0.546.0), plotly.js (3.1.1→3.1.2), recharts (3.2.1→3.3.0), tailwindcss (4.1.14→4.1.15), and rolldown-vite (7.1.17→7.1.19)
- New Icon: Added
monitor_cloudto the Lucide icon list - Lock Files: Updated
uv.lockwith transitive dependency updates
Issues Found
monitor_cloudicon is placed out of alphabetical order inLUCIDE_ICON_LIST
Confidence Score: 4/5
- This PR is mostly safe to merge with one minor ordering issue that needs correction
- The PR contains routine dependency updates which are generally low-risk. However, the
monitor_cloudicon is incorrectly positioned in the alphabetically-sortedLUCIDE_ICON_LIST, which could cause the icon lookup logic to malfunction. This is a simple fix but important for correctness. - Pay attention to
reflex/components/lucide/icon.py- the icon list ordering must be corrected
Important Files Changed
File Analysis
| Filename | Score | Overview |
|---|---|---|
| pyproject.toml | 5/5 | Version bump to 0.8.17dev1, updates reflex-hosting-cli, wrapt upper bound, and ruff pre-commit hook version |
| reflex/components/lucide/icon.py | 3/5 | Updates lucide-react to 0.546.0 and adds monitor_cloud icon, but icon is incorrectly positioned in the list |
| reflex/components/plotly/plotly.py | 5/5 | Updates all plotly.js dependencies from 3.1.1 to 3.1.2 across all variants |
| reflex/components/recharts/recharts.py | 5/5 | Updates recharts library from 3.2.1 to 3.3.0 |
| reflex/plugins/tailwind_v4.py | 5/5 | Updates tailwindcss from 4.1.14 to 4.1.15 and @tailwindcss/postcss to matching version |
Sequence Diagram
sequenceDiagram
participant Dev as Developer
participant PM as Package Manager (uv)
participant NPM as NPM Registry
participant App as Reflex App
Dev->>PM: Bump version to 0.8.17dev1
Dev->>PM: Update Python dependencies
Note over PM: reflex-hosting-cli: 0.1.55→0.1.57<br/>wrapt: <2.0→<3.0<br/>ruff: v0.14.0→v0.14.1
Dev->>NPM: Update Frontend Libraries
Note over NPM: lucide-react: 0.545.0→0.546.0<br/>plotly.js: 3.1.1→3.1.2<br/>recharts: 3.2.1→3.3.0<br/>tailwindcss: 4.1.14→4.1.15<br/>rolldown-vite: 7.1.17→7.1.19
Dev->>App: Add new icon (monitor_cloud)
PM->>PM: Update uv.lock with new dependencies
Note over PM: fastapi, iniconfig, narwhals,<br/>psutil, psycopg updated
App->>App: Regenerate pyi_hashes.json
Note over App: Hash updated for icon.pyi
7 files reviewed, 1 comment
CodSpeed Performance ReportMerging #5916 will create unknown performance changesComparing Summary
Footnotes
|
No description provided.