-
Notifications
You must be signed in to change notification settings - Fork 74
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
deprecate system.mem.used.all metric #485
Conversation
✅ Deploy Preview for agent-public-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #485 +/- ##
==========================================
- Coverage 66.29% 66.09% -0.21%
==========================================
Files 118 118
Lines 13441 13439 -2
==========================================
- Hits 8911 8882 -29
- Misses 3936 3964 +28
+ Partials 594 593 -1
☔ View full report in Codecov by Sentry. |
Proposed changes
Deprecate system.mem.used.all metric and remove system.mem.used.all from tests
Fixed system.mem.used to have the correct value
Checklist
Before creating a PR, run through this checklist and mark each as complete.
CONTRIBUTING
documentmake install-tools
and have attached any dependency changes to this pull requestREADME.md
)