-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Rewrite #63 on top of the new master with the tests that already exist there. #468
Rewrite #63 on top of the new master with the tests that already exist there. #468
Conversation
Critic review: https://critic.hoppipolla.co.uk/r/467 This is an external review system which you may optionally use for the code review of your pull request. In order to help critic track your changes, please do not make in-place history rewrites (e.g. via |
- Correct support.js reference path - Replace IDBTransaction.READ_WRITE with "readwrite" as it is obsolete - Remove console.log - Enable commented assertions
as it is duplicate to idbcursor_continue_index4.htm
Squashed and landed. |
Nice :) We should maybe put in authorship information to these files. They were created before we required that, but I think it might make sense to put it in again after the fact. I don't know who specifically at Microsoft that'd be, but it'd be possible to at least write Microsoft in there. |
No description provided.