refactor: remove helpers deprecated for more than 3y#810
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #810 +/- ##
==========================================
+ Coverage 92.05% 92.25% +0.19%
==========================================
Files 32 32
Lines 4216 4208 -8
==========================================
+ Hits 3881 3882 +1
+ Misses 256 247 -9
Partials 79 79
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR removes deprecated helper functions that have been marked as deprecated for more than 3 years. The changes clean up the codebase by removing wrapper functions that were kept for backward compatibility but are no longer needed.
Changes:
- Removed
Elipsefunction fromstring.go(typo/deprecated alias forEllipsis) - Removed
Batch,BatchWithTimeout, andChannelMergefunctions fromchannel.go(deprecated aliases forBuffer,BufferWithTimeout, andFanInrespectively)
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| string.go | Removes the deprecated Elipse function, a misspelled alias for Ellipsis |
| channel.go | Removes three deprecated channel helper functions: Batch (alias for Buffer), BatchWithTimeout (alias for BufferWithTimeout), and ChannelMerge (alias for FanIn) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.