Skip to content

Conversation

@callmehiphop
Copy link
Contributor

This fixes 2 things:

  1. The old documentation for creating clusters went missing, so updates were made to several links
  2. Certain options were not being preserved when not using default application credentials.

@callmehiphop callmehiphop added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. api: bigtable Issues related to the Bigtable API. labels Jul 12, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 12, 2016
@callmehiphop
Copy link
Contributor Author

@lesv we noticed that one can optionally provide an API key instead of a keyfile or default application creds. Is there a standard way of providing said key?

@stephenplusplus
Copy link
Contributor

Throwing a test on here would be good.

@coveralls
Copy link

coveralls commented Jul 12, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 2b31d15 on callmehiphop:bigtable-auth-fixes into 96eb260 on GoogleCloudPlatform:master.

fakeUtil.normalizeArguments = normalizeArguments;
});

it('should leave the original options unaltered', function() {

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@coveralls
Copy link

coveralls commented Jul 12, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling ba265a8 on callmehiphop:bigtable-auth-fixes into 96eb260 on GoogleCloudPlatform:master.

@stephenplusplus stephenplusplus merged commit 3f01f61 into googleapis:master Jul 12, 2016
miguelvelezsa pushed a commit that referenced this pull request Jul 23, 2025
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [protobufjs-cli](https://togithub.com/protobufjs/protobuf.js) | [`1.1.0` -> `1.1.1`](https://renovatebot.com/diffs/npm/protobufjs-cli/1.1.0/1.1.1) | [![age](https://badges.renovateapi.com/packages/npm/protobufjs-cli/1.1.1/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/protobufjs-cli/1.1.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/protobufjs-cli/1.1.1/compatibility-slim/1.1.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/protobufjs-cli/1.1.1/confidence-slim/1.1.0)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>protobufjs/protobuf.js</summary>

### [`v1.1.1`](https://togithub.com/protobufjs/protobuf.js/releases/tag/protobufjs-cli-v1.1.1): protobufjs-cli: v1.1.1

[Compare Source](https://togithub.com/protobufjs/protobuf.js/compare/protobufjs-cli-v1.1.0...protobufjs-cli-v1.1.1)

##### Bug Fixes

-   **cli:** fix relative path to Google pb files ([#&#8203;1859](https://togithub.com/protobufjs/protobuf.js/issues/1859)) ([e42eea4](https://togithub.com/protobufjs/protobuf.js/commit/e42eea4868b11f4a07934804a56683321ed191e2))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 9am and before 3pm" (UTC), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/gax-nodejs).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNC4xMTQuMCIsInVwZGF0ZWRJblZlciI6IjM0LjExNC4wIn0=-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtable Issues related to the Bigtable API. cla: yes This human has signed the Contributor License Agreement. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants