-
Notifications
You must be signed in to change notification settings - Fork 301
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
chore: upgrade all dependencies #2996
Conversation
Codecov ReportBase: 52.95% // Head: 52.99% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #2996 +/- ##
==========================================
+ Coverage 52.95% 52.99% +0.04%
==========================================
Files 338 334 -4
Lines 52099 51921 -178
==========================================
- Hits 27587 27517 -70
+ Misses 22897 22793 -104
+ Partials 1615 1611 -4
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
046ac98
to
4baf2a7
Compare
Co-authored-by: Aris Tzoumas <atzoumas@rudderstack.com>
…der-server into chore.upgradeDependencies
@@ -53,5 +53,6 @@ func (rateLimiter *HandleT) SetUp() { | |||
|
|||
// LimitReached returns true if number of events in the rolling window is less than the max events allowed, else false | |||
func (rateLimiter *HandleT) LimitReached(key string) bool { | |||
return rateLimiter.restrictor.LimitReached(key) | |||
ok, _ := rateLimiter.restrictor.LimitReached(key) |
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.
Note: we should add a task to move away from github.com/EagleChen/restrictor
since we are already using github.com/throttled/throttled/v2
elsewhere
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.
Added one here
Please add an appropriate description for this pull request |
Description
This PR aims to upgrade the Go minor version to
1.20.1
, Alpine version to3.17
, GolangCI Lint version to1.51.1
and also the dependent packages in go.mod fileNotion Ticket
Notion Link
Security