Skip to content

Conversation

@xelandernt
Copy link
Collaborator

Goes through the points in #142:

  • Tested 2.0.0a3 with simple FastAPI application with database, but at least partially covering most of the new features.
  • Add a warning when overriding existing container (only relevant for string providers)
  • Change the default of preserve_global_context in container_context() and DiContextMiddleware to true since a "global" context should not be so easy to reset.
  • Fix documentation (some sections and especially enumerated lists do not show up correctly).
  • Expand the documentation of some sections for clarity

@xelandernt xelandernt self-assigned this Feb 9, 2025
@xelandernt xelandernt added documentation Improvements or additions to documentation enhancement New feature or request labels Feb 9, 2025
@xelandernt xelandernt added this to the 2.0.0 milestone Feb 9, 2025
@xelandernt xelandernt requested a review from lesnik512 February 9, 2025 21:11
@xelandernt
Copy link
Collaborator Author

@lesnik512 As mentioned above, I did a prelim test on a relatively simple application (similar to the example applications). I load tested it with ~150 RPS and did not have any issues.

@lesnik512 lesnik512 merged commit 5389c4a into modern-python:main Feb 10, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants