-
Notifications
You must be signed in to change notification settings - Fork 143
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
Usage of 'mozIndexedDB' should be a warning instead of an error #797
Conversation
Please update r+wc |
I get some (seemlingly) unrelated test failures, let's wait for travis... |
Afaics, the rules document is correct, just the implementation was wrong. |
Right on. A nitpick would be to change the rule from "not allowed" to "discouraged". |
I am totally at loss about the test failures. Could you give me a hint? Also, I don't see where it says "not allowed". |
6443bc6
to
9fcee68
Compare
Ok, I updated the rule description. |
After reviewing this and the tests failures this is r- (not safe to merge) until #798 is fixed. After that it's r+ but I'll have a look at making sure it's all okay and merge it in myself 👍 |
Fixes #796