-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
chore(main): release 0.0.59 #971
chore(main): release 0.0.59 #971
Conversation
dcec9c5
to
0abfd04
Compare
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
0abfd04
to
cd9fd1f
Compare
cd9fd1f
to
cc1c121
Compare
cc1c121
to
d2d6fb3
Compare
d2d6fb3
to
1ed6ca6
Compare
1ed6ca6
to
5f8b28a
Compare
Risk Level 2 - /home/runner/work/quivr/quivr/backend/core/main.py The code seems to be well-structured and follows the SOLID principles. However, there are a few areas that could be improved for better readability and maintainability:
Here's an example of how you might implement these changes: # Config class to handle environment variables
class Config:
SENTRY_DSN: str = os.getenv(\"SENTRY_DSN\")
# Simplified error handling
@app.exception_handler(RequestValidationError)
async def validation_exception_handler(request: Request, exc: RequestValidationError):
exc_str = str(exc).strip()
logger.error(request, exc_str)
content = {
\"status_code\": status.HTTP_422_UNPROCESSABLE_ENTITY,
\"message\": exc_str,
\"data\": None,
}
return JSONResponse(content=content, status_code=status.HTTP_422_UNPROCESSABLE_ENTITY) Remember to replace all instances of 🌍💼🔧 Powered by Code Review GPT |
5f8b28a
to
9cd5d81
Compare
a644571
to
d63b0a2
Compare
d63b0a2
to
3783553
Compare
3783553
to
24cf1b2
Compare
24cf1b2
to
5653c33
Compare
5653c33
to
ae27708
Compare
🤖 Release is at https://github.com/StanGirard/quivr/releases/tag/v0.0.59 🌻 |
🤖 I have created a release beep boop
0.0.59 (2023-08-20)
Features
Bug Fixes
This PR was generated with Release Please. See documentation.