Skip to content

Update IDBDatabase interface - #36

Merged
Zhengbo Li (zhengbli) merged 1 commit into
microsoft:masterfrom
falsandtru:patch-1
Jan 11, 2016
Merged

Zhengbo Li (zhengbli) merged 1 commit into
microsoft:masterfrom
falsandtru:patch-1

Conversation

@falsandtru

Copy link
Copy Markdown
Contributor

@msftclas

Hi falsandtru (@falsandtru), I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@falsandtru

Copy link
Copy Markdown
Contributor Author

ouch

@msftclas

falsandtru (@falsandtru), Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.

Thanks, MSBOT;

@ghost

Copy link
Copy Markdown

Is there any issue tracking this in TypeScript repo? A link to it would be nice.
This PR needs to be squashed to a single commit for multiple reasons. So:

cd TSJS-lib-generator
git rebase -i HEAD~4
# editor will open
# replace `pick` with `f`, for all lines except the first one, save and close the editor
git push -f

@falsandtru

Copy link
Copy Markdown
Contributor Author

I didn't report this issue in TypeScript repo.

@mhegazy

Copy link
Copy Markdown
Contributor

falsandtru (@falsandtru) thanks for the PR, two comments:

  1. please file an issue on the TS repo, this will allow for documenting the change.
  2. do not change the webidl file directly, this file is external. put your changes in https://github.com/Microsoft/TSJS-lib-generator/blob/master/inputfiles/overridingTypes.json instead.

@falsandtru

Copy link
Copy Markdown
Contributor Author

All right, I'll fix it. And now, is there a way to fix webidl files?

@mhegazy

Copy link
Copy Markdown
Contributor

The webidl files are external. we grab them periodically from the IE/Edge team. files inputfiles/addedTypes.json, inputfiles/overridingTypes.json, and inputfiles/removedTypes.json allow to augment the webidl definitions to get the desired shape. this way when a new version of the webidl is picked up the augmentations are not lost.

See Readme.md for more information.

@falsandtru

Copy link
Copy Markdown
Contributor Author

I understood that. Thanks.

@falsandtru

Copy link
Copy Markdown
Contributor Author

Mohamed Hegazy (@mhegazy) I filed a new issue: microsoft/TypeScript#6408, and I fixed your specified points. Please review, Mohamed Hegazy (@mhegazy) Zhengbo Li (@zhengbli) .

@falsandtru

Copy link
Copy Markdown
Contributor Author

I have some unpublished fixes. I want to send that fixes after accepting this commit.

@mhegazy

Copy link
Copy Markdown
Contributor

👍

Zhengbo Li (zhengbli) pushed a commit that referenced this pull request Jan 11, 2016
Update `IDBDatabase` interface
@zhengbli
Zhengbo Li (zhengbli) merged commit 9951366 into microsoft:master Jan 11, 2016
@zhengbli

Copy link
Copy Markdown

falsandtru (@falsandtru) Thank you for contributing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants