Skip to content
This repository was archived by the owner on Apr 3, 2024. It is now read-only.

Add types to all non-test files#291

Merged
DominicKramer merged 3 commits intogoogleapis:masterfrom
DominicKramer:to-ts-prs/5-add-types
Jun 29, 2017
Merged

Add types to all non-test files#291
DominicKramer merged 3 commits intogoogleapis:masterfrom
DominicKramer:to-ts-prs/5-add-types

Conversation

@DominicKramer
Copy link
Copy Markdown
Contributor

No description provided.

@DominicKramer DominicKramer requested a review from ofrobots June 28, 2017 19:55
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 28, 2017
@kjin kjin removed their assignment Jun 28, 2017
@kjin kjin self-requested a review June 28, 2017 23:03
Copy link
Copy Markdown
Contributor

@ofrobots ofrobots left a comment

Choose a reason for hiding this comment

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

LGTM w/ nit & a question.

static createDebuggee(projectId, uid, serviceContext, sourceContext, description,
errorMessage, onGCP) {
// TODO: Determine the type of sourceContext
static createDebuggee(projectId: string, uid: string, serviceContext: { service?: string, version?: string, minorVersion_?: string }, sourceContext: { [key: string]: string }, description: string,

This comment was marked as spam.

This comment was marked as spam.

* @private
*/
start() {
start(): void {

This comment was marked as spam.

This comment was marked as spam.

@DominicKramer DominicKramer merged commit 675ef72 into googleapis:master Jun 29, 2017
@DominicKramer DominicKramer deleted the to-ts-prs/5-add-types branch July 6, 2017 20:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants