Skip to content
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

improve types (2) #356

Merged
merged 7 commits into from
Nov 28, 2018
Merged

improve types (2) #356

merged 7 commits into from
Nov 28, 2018

Conversation

jkwlui
Copy link
Member

@jkwlui jkwlui commented Nov 21, 2018

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 21, 2018
@@ -32,7 +30,7 @@ import { CallOptions } from 'google-gax';
* @param {object} apiResponse The full API response.
*/
export interface GetPolicyCallback {
(err?: Error|null, acl?: Policy, apiResponse?: r.Response): void;

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@JustinBeckwith
Copy link
Contributor

@kinwa91 I am so sorry for what happened here.

src/iam.ts Outdated Show resolved Hide resolved
src/iam.ts Outdated Show resolved Hide resolved
src/index.ts Show resolved Hide resolved
src/index.ts Show resolved Hide resolved
src/index.ts Show resolved Hide resolved
src/index.ts Outdated
* Typed any since it's importing Javascript source.
*/
// tslint:disable-next-line:no-any
(err: Error|null, gaxClient?: any): void;

This comment was marked as spam.

This comment was marked as spam.

Copy link
Contributor

@JustinBeckwith JustinBeckwith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM after the failing sample tests are addressed

@jkwlui jkwlui merged commit 05ff4ca into master Nov 28, 2018
@jkwlui jkwlui deleted the no-implicit-any-2 branch November 28, 2018 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants