Use class-name-suffix for parser config TypedDicts#2888
Conversation
📝 WalkthroughWalkthroughThis PR renames parser configuration TypedDict classes to include a "Dict" suffix for consistency, updates corresponding base class relationships, refactors type imports to use new names directly, and adds a matching pyproject.toml configuration key. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 📒 Files selected for processing (3)
🧰 Additional context used🧬 Code graph analysis (1)src/datamodel_code_generator/_types/__init__.py (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (12)
🔇 Additional comments (5)
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 |
|
📚 Docs Preview: https://pr-2888.datamodel-code-generator.pages.dev |
CodSpeed Performance ReportMerging #2888 will improve performance by 21.86%Comparing
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | WallTime | test_perf_duplicate_names |
999.8 ms | 841.4 ms | +18.82% |
| ⚡ | WallTime | test_perf_large_models_pydantic_v2 |
3.6 s | 3 s | +20.28% |
| ⚡ | WallTime | test_perf_complex_refs |
2.1 s | 1.7 s | +21.86% |
| ⚡ | WallTime | test_perf_deep_nested |
6 s | 5.1 s | +18.14% |
| ⚡ | WallTime | test_perf_openapi_large |
2.9 s | 2.5 s | +18.07% |
| ⚡ | WallTime | test_perf_graphql_style_pydantic_v2 |
811 ms | 690.6 ms | +17.43% |
| ⚡ | WallTime | test_perf_kubernetes_style_pydantic_v2 |
2.6 s | 2.2 s | +19.45% |
| ⚡ | WallTime | test_perf_multiple_files_input |
3.7 s | 3.1 s | +18.94% |
| ⚡ | WallTime | test_perf_aws_style_openapi_pydantic_v2 |
1.9 s | 1.6 s | +17.31% |
| ⚡ | WallTime | test_perf_stripe_style_pydantic_v2 |
2 s | 1.6 s | +19.9% |
| ⚡ | WallTime | test_perf_all_options_enabled |
6.7 s | 5.7 s | +17.35% |
Footnotes
-
98 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2888 +/- ##
=======================================
Coverage 99.37% 99.37%
=======================================
Files 92 92
Lines 16121 16121
Branches 1898 1898
=======================================
Hits 16020 16020
Misses 52 52
Partials 49 49
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Breaking Change AnalysisResult: No breaking changes detected Reasoning: This PR is a pure internal refactoring. The changes rename internal TypedDict classes in the auto-generated This analysis was performed by Claude Code Action |
|
🎉 Released in 0.52.0 This PR is now available in the latest release. See the release notes for details. |
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.