Remove unused numeric throttle classses
zackkrida opened this issue · 2 comments
zackkrida commented
Problem
As discussed in the comments of #864, we should remove the unused throttle classes OneThousandPerMinute and OnePerSecond.
Description
Removing unused code.
Alternatives
Additional context
Implementation
- 🙋 I would be interested in implementing this feature.
AetherUnbound commented
Unfortunately, these classes are all presently being used:
TenPerDay
OneThousandPerMinute
OnePerSecond
Perhaps they could be replaced with other throttle classes we have now, I'm not familiar enough with the throttling setup to be able to tell at this point.
krysal commented
Excellent find @AetherUnbound! I'll close the issue as we can always open another one if a clear need arises.