Added @example tags to src/table.js#239
Conversation
|
@vijay-qlogic: Please fix the errors. |
Codecov Report
@@ Coverage Diff @@
## master #239 +/- ##
=======================================
Coverage ? 100%
=======================================
Files ? 10
Lines ? 1281
Branches ? 0
=======================================
Hits ? 1281
Misses ? 0
Partials ? 0
Continue to review full report at Codecov.
|
|
@vijay-qlogic, please fix lint. |
3dad6f6 to
78dde87
Compare
6cb5eba to
adfdcc6
Compare
|
@vijay-qlogic, can you please resolve the conflicts? |
f076d72 to
48e27bb
Compare
|
@sduskis conflicts resolved |
92bdaa0 to
2ab64bb
Compare
|
The samples should be basically what the comments were. They should not be full examples that include printlns. |
a17c3f1 to
94bd339
Compare
5a3edc0 to
85123d5
Compare
9ff8eac to
94d8a06
Compare
|
|
||
| // creates a new Instance | ||
| instance | ||
| await instance |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
| const snippets = { | ||
| createInstance: (instanceId, clusterId, callback) => { | ||
| // [START bigtable_create_instance] | ||
| createInstance: async (instanceId, clusterId, callback) => { |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
dbc82b2 to
a2ed241
Compare
a2ed241 to
462f3b7
Compare
sduskis
left a comment
There was a problem hiding this comment.
@vijay-qlogic, please fix the conflicts
552426c to
19e7ada
Compare
| callback(err); | ||
| return; | ||
| // Handle the error. | ||
| return callback(err); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
sduskis
left a comment
There was a problem hiding this comment.
We need a few minor changes, but otherwise looks great.
| const Bigtable = require('@google-cloud/bigtable'); | ||
| const bigtable = new Bigtable(); | ||
| const instance = bigtable.instance(instanceId); | ||
| const caughtError = false; |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
| const Bigtable = require('@google-cloud/bigtable'); | ||
| const bigtable = new Bigtable(); | ||
| const instance = bigtable.instance(instanceId); | ||
| const caughtError = false; |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
| const Bigtable = require('@google-cloud/bigtable'); | ||
| const bigtable = new Bigtable(); | ||
| const instance = bigtable.instance(instanceId); | ||
| const caughtError = false; |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Always using `require` in the snippet, and including it within the tag.
8287a1f to
bdfc711
Compare
Fixes #157 for table.js (it's a good idea to open an issue first for discussion)