Skip to content

Conversation

@stephenplusplus
Copy link
Contributor

Fixes #1422

@stephenplusplus stephenplusplus added the api: bigquery Issues related to the BigQuery API. label Sep 2, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 2, 2016
@callmehiphop
Copy link
Contributor

Looks like there are a few linting issues per Travis.

@stephenplusplus
Copy link
Contributor Author

That was an issue we merged to master. It's fixed now!

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 82f293b on stephenplusplus:spp--1422 into bf9929e on GoogleCloudPlatform:master.

@stephenplusplus
Copy link
Contributor Author

stephenplusplus commented Sep 6, 2016

Instead of blitzing Travis, I just ran against Node stable. Feel free to restart the others.

@callmehiphop
Copy link
Contributor

Should there be some sort of documentation/example stating we support nested schemas? Or was it already assumed/documented and we just didn't implement it correctly..?

@stephenplusplus
Copy link
Contributor Author

It's more like an implementation detail that only matters if the user didn't know to put it in the right place. Kind of like an API that accepts a "jsonPayload" or "textPayload"-- we detect "did we get an object? Okay, send this as jsonPayload". Doesn't matter to the user, they just expect us to do the right thing.

@callmehiphop
Copy link
Contributor

Gotcha

@callmehiphop callmehiphop merged commit 2d72e01 into googleapis:master Sep 7, 2016
miguelvelezsa pushed a commit that referenced this pull request Jul 23, 2025
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigquery Issues related to the BigQuery API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants