Update gapi.load typing.

[trshafer]

Please fill in this template.

• Use a meaningful title for the pull request. Include the name of the package modified.
• Test the change in your own code. (Compile and run.)
• Follow the advice from the readme.
• Avoid common mistakes.
• Run npm run lint package-name (or tsc if no tslint.json is present).

If changing an existing definition:

• Provide a URL to documentation or source code which provides context for the suggested changes: https://developers.google.com/api-client-library/javascript/reference/referencedocs#gapiloadlibraries-callbackorconfig
• Increase the version number in the header if appropriate.
• If you are making substantial changes, consider adding a tslint.json containing { "extends": "dtslint/dt.json" }.

[dt-bot]

types/gapi/index.d.ts

to author (@sgtfrankieboy). Could you review this PR?
👍 or 👎?

Checklist

• pass the Travis CI test?

[trshafer]

I know this is kind of a weird declaration, () => void is technically correct. However many consumers use something like:

new Promise((resolve) => {
  gapi.load('thing', resolve)
})

and resolve does not have the type of () => void, so I added Function as a blanket catchall.

[mhegazy]

If this is a callback, then (...args:any[]) => void should be your fallback. or just use Function. the union does not make much sense, since Function subsumes it.

[trshafer]

Done. Removed Function and changed the fallback to (...args: any[]) => void.

[mhegazy]

If this is a callback, then (...args:any[]) => void should be your fallback. or just use Function. the union does not make much sense, since Function subsumes it.

[mhegazy]

you do not need ()=>void

[trshafer]

Done. the load callback is now only (...args: any[]) => void;