Skip to content

Conversation

@samtstern
Copy link
Contributor

No description provided.

@samtstern samtstern requested a review from hiranya911 August 15, 2018 16:17
@samtstern
Copy link
Contributor Author

99% sure the Travis failures are unrelated since this PR only adds a JSON file in a previously unknown directory.

Copy link
Contributor

@hiranya911 hiranya911 left a comment

Choose a reason for hiding this comment

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

LGTM

"firebase/firebase-admin-go",
"firebase/firebase-admin-python"
]
} No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: add new line at eof

@hiranya911
Copy link
Contributor

Build break is due to DefinitelyTyped/DefinitelyTyped#27943, a breaking change which they released as 1.7.1 :(

@hiranya911
Copy link
Contributor

Build will be back to normal once #340 is merged.

@samtstern samtstern merged commit 333f6f7 into firebase:master Aug 15, 2018
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.

2 participants