Add architecture documentation#3464
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughAdds a new ChangesArchitecture Documentation Auto-Generation
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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-3464.datamodel-code-generator.pages.dev |
Merging this PR will improve performance by 15.63%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | WallTime | test_perf_large_models_pydantic_v2 |
3.7 s | 3.1 s | +17.31% |
| ⚡ | WallTime | test_perf_complex_refs |
2 s | 1.7 s | +17.01% |
| ⚡ | WallTime | test_perf_multiple_files_input |
3.6 s | 3.1 s | +16.09% |
| ⚡ | WallTime | test_perf_deep_nested |
5.2 s | 4.5 s | +15.98% |
| ⚡ | WallTime | test_perf_duplicate_names |
1,084.1 ms | 936.4 ms | +15.77% |
| ⚡ | WallTime | test_perf_aws_style_openapi_pydantic_v2 |
1.9 s | 1.7 s | +15.61% |
| ⚡ | WallTime | test_perf_openapi_large |
2.9 s | 2.5 s | +15.24% |
| ⚡ | WallTime | test_perf_graphql_style_pydantic_v2 |
806.7 ms | 700.6 ms | +15.14% |
| ⚡ | WallTime | test_perf_all_options_enabled |
5.4 s | 4.7 s | +15.1% |
| ⚡ | WallTime | test_perf_stripe_style_pydantic_v2 |
2 s | 1.8 s | +14.43% |
| ⚡ | WallTime | test_perf_kubernetes_style_pydantic_v2 |
2.6 s | 2.3 s | +14.24% |
Tip
Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.
Comparing docs/architecture-overview (ba5159e) with main (8abb86c)2
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. ↩
-
No successful run was found on
main(3dd90d7) during the generation of this report, so 8abb86c was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3464 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 148 149 +1
Lines 32156 32176 +20
Branches 3767 3767
=========================================
+ Hits 32156 32176 +20
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Breaking Change AnalysisResult: No breaking changes detected Reasoning: PR #3464 is purely additive documentation/tooling. It adds a new docs/architecture.md page, a new scripts/build_architecture_docs.py helper that generates an inventory section from source, an accompanying test and expected-output fixtures, and registers the new page/script in zensical.toml nav, the generated-docs-sync workflow, tox.ini, README.md, and the llms.txt/llms-full.txt docs. The deleted-lines.txt file is empty, confirming no lines were removed. Nothing changes generated code output, Jinja2 templates, CLI options, the Python API, default values/behavior, supported Python versions, or error handling. Therefore there are no breaking changes for end users. This analysis was performed by Claude Code Action |
|
🎉 Released in 0.65.0 This PR is now available in the latest release. See the release notes for details. |
Summary by CodeRabbit