-
Notifications
You must be signed in to change notification settings - Fork 51
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
feat(lib): remove version check of resty.events #162
Conversation
ec1614e
to
580ae05
Compare
580ae05
to
2375c2c
Compare
2375c2c
to
96342f3
Compare
96342f3
to
dbdce63
Compare
b8434b4
to
554406d
Compare
554406d
to
e84e0d9
Compare
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.
The changes LGTM, but wouldn't it be better if we improved the cache key instead of removing the cache at all?
Perhaps with time. My conclusion is that we have lost more time to caching issues that it has ever given time back. There is always a case that someone who turns on caches doesn't seem to think about. Caching is totally unneeded on low traffic repo like this. Where full anything takes just minutes. |
Summary
lua-resty-events is still in active development, we should support more possible versions.
KAG-4760