-
Notifications
You must be signed in to change notification settings - Fork 580
ci: Update tox and pin packaging version for tox #5381
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
Conversation
Semver Impact of This PR🟢 Patch (bug fixes) 📋 Changelog PreviewThis is how your changes will appear in the changelog. New Features ✨Span Streaming
Bug Fixes 🐛Span Streaming
Other
Internal Changes 🔧Fastmcp
Mcp
Other
🤖 This preview updates automatically when you update the PR. |
sentrivana
left a comment
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.
Thanks for pinning. Looks like new mcp fails the test suite though.
I'll look into removing the pin in the meantime. We're misusing the Version class a bit, so that will need updating.
|
|
Codecov Results 📊✅ 56 passed | Total: 56 | Pass Rate: 100% | Execution Time: 13.84s All tests are passing successfully. ✅ Patch coverage is 100.00%. Project has 12683 uncovered lines. Files with missing lines (181)
Generated by Codecov Action |
Description
Fix exception when running
populate_tox.pyin CI by pinning packaging. See https://github.com/getsentry/sentry-python/actions/runs/21345873752.Bump
actions/checkoutin the test group template to match current./githubconfig files.Issues
Reminders
tox -e linters.feat:,fix:,ref:,meta:)