Skip to content
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

Remove flawed RateLimit implementation. #16220

Merged
merged 1 commit into from Feb 23, 2019

Conversation

Projects
None yet
3 participants
@pchote
Copy link
Member

pchote commented Feb 20, 2019

Fixes #16195.

@pchote pchote added this to the Next Release milestone Feb 20, 2019

@chrisforbes

This comment has been minimized.

Copy link
Member

chrisforbes commented Feb 23, 2019

Actually fixing this would be better than ripping it out?

@chrisforbes

This comment has been minimized.

Copy link
Member

chrisforbes commented Feb 23, 2019

For stable: +1.

@pchote pchote merged commit ea05e8a into OpenRA:bleed Feb 23, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@pchote pchote deleted the pchote:remove-notification-rate-limit branch Feb 23, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.