Skip to content
This repository was archived by the owner on Dec 19, 2023. It is now read-only.

Enable noImplicitAny#105

Merged
jkwlui merged 2 commits intogoogleapis:masterfrom
JustinBeckwith:noImplicitAny
Sep 14, 2018
Merged

Enable noImplicitAny#105
jkwlui merged 2 commits intogoogleapis:masterfrom
JustinBeckwith:noImplicitAny

Conversation

@JustinBeckwith
Copy link
Copy Markdown
Contributor

@JustinBeckwith JustinBeckwith commented Sep 11, 2018

Fixes #<issue_number_goes_here> (it's a good idea to open an issue first for discussion)

  • Tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 11, 2018
@ghost ghost assigned JustinBeckwith Sep 11, 2018
@JustinBeckwith
Copy link
Copy Markdown
Contributor Author

JustinBeckwith commented Sep 13, 2018

gentle ping @kinwa91

@jkwlui jkwlui merged commit 46533f2 into googleapis:master Sep 14, 2018
@JustinBeckwith JustinBeckwith mentioned this pull request Sep 14, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@callmehiphop callmehiphop Awaiting requested review from callmehiphop

@stephenplusplus stephenplusplus Awaiting requested review from stephenplusplus

1 more reviewer

@jkwlui jkwlui jkwlui approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@JustinBeckwith JustinBeckwith

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@JustinBeckwith @jkwlui @googlebot