-
Notifications
You must be signed in to change notification settings - Fork 44.4k
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
security(platform/backend): Add health endpoint #8341
Conversation
PR Reviewer Guide πHere are some key observations to aid the review process:
|
β Deploy Preview for auto-gpt-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests β
Additional details and impacted files@@ Coverage Diff @@
## dev #8341 +/- ##
=======================================
Coverage 49.62% 49.62%
=======================================
Files 144 144
Lines 8916 8916
Branches 1240 1240
=======================================
Hits 4425 4425
Misses 4344 4344
Partials 147 147
Flags with carried forward coverage won't be shown. Click here to find out more. β View full report in Codecov by Sentry. |
Fix linting but code is good. Now that we have this, it may be a good place to add other info for the bot to ping like version etc cc @Bentlybro |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just need to fix the linting errors
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Conflicts have been resolved! π A maintainer will review the pull request shortly. |
* add health endpoint * fix linting --------- Co-authored-by: Nicholas Tindle <[email protected]>
Background
Add health endpoint to use instead of docs
Changes ποΈ
Added a health endpoint to rest api
Testing π
Note
Only for the new autogpt platform, currently in autogpt_platform/