Skip to content
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

Fixed documentation and adjusted API for 2.0 release. #156

Merged
merged 23 commits into from
Feb 10, 2025

Conversation

alexanderlazarev0
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

@alexanderlazarev0 alexanderlazarev0 self-assigned this Feb 9, 2025
@alexanderlazarev0 alexanderlazarev0 added documentation Improvements or additions to documentation enhancement New feature or request labels Feb 9, 2025
@alexanderlazarev0 alexanderlazarev0 added this to the 2.0.0 milestone Feb 9, 2025
@alexanderlazarev0
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