Skip to content
This repository was archived by the owner on Nov 18, 2025. It is now read-only.

fix: refactor createByteLengthFunction, warn if fails#1331

Merged
alexander-fenster merged 2 commits intomainfrom
warn-on-serialization-failure
Aug 29, 2022
Merged

fix: refactor createByteLengthFunction, warn if fails#1331
alexander-fenster merged 2 commits intomainfrom
warn-on-serialization-failure

Conversation

@alexander-fenster
Copy link
Contributor

  1. Warn (just once) if a function returned by createByteLengthFunction fails.
  2. Make createByteLengthFunction available for fallback scenarios.
  3. Use the proper type for createByteLengthFunction.

@alexander-fenster alexander-fenster requested a review from a team as a code owner August 29, 2022 20:57
@product-auto-label product-auto-label bot added the size: s Pull request size is small. label Aug 29, 2022
@alexander-fenster alexander-fenster merged commit d27f7fa into main Aug 29, 2022
@alexander-fenster alexander-fenster deleted the warn-on-serialization-failure branch August 29, 2022 21:16
gcf-merge-on-green bot pushed a commit that referenced this pull request Aug 29, 2022
🤖 I have created a release *beep* *boop*
---


## [3.3.1](v3.3.0...v3.3.1) (2022-08-29)


### Bug Fixes

* **deps:** update dependency protobufjs to v7.1.0 ([#1329](#1329)) ([fc5446e](fc5446e))
* **deps:** update dependency protobufjs-cli to v1.0.1 ([#1328](#1328)) ([b810d37](b810d37))
* refactor createByteLengthFunction, warn if fails ([#1331](#1331)) ([d27f7fa](d27f7fa))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
miguelvelezsa pushed a commit to googleapis/google-cloud-node that referenced this pull request Jul 23, 2025
🤖 I have created a release *beep* *boop*
---


## [3.3.1](googleapis/gax-nodejs@v3.3.0...v3.3.1) (2022-08-29)


### Bug Fixes

* **deps:** update dependency protobufjs to v7.1.0 ([#1329](googleapis/gax-nodejs#1329)) ([fc5446e](googleapis/gax-nodejs@fc5446e))
* **deps:** update dependency protobufjs-cli to v1.0.1 ([#1328](googleapis/gax-nodejs#1328)) ([b810d37](googleapis/gax-nodejs@b810d37))
* refactor createByteLengthFunction, warn if fails ([#1331](googleapis/gax-nodejs#1331)) ([d27f7fa](googleapis/gax-nodejs@d27f7fa))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
miguelvelezsa pushed a commit to googleapis/google-cloud-node-core that referenced this pull request Oct 22, 2025
🤖 I have created a release *beep* *boop*
---


## [3.3.1](googleapis/gax-nodejs@v3.3.0...v3.3.1) (2022-08-29)


### Bug Fixes

* **deps:** update dependency protobufjs to v7.1.0 ([#1329](googleapis/gax-nodejs#1329)) ([fc5446e](googleapis/gax-nodejs@fc5446e))
* **deps:** update dependency protobufjs-cli to v1.0.1 ([#1328](googleapis/gax-nodejs#1328)) ([b810d37](googleapis/gax-nodejs@b810d37))
* refactor createByteLengthFunction, warn if fails ([#1331](googleapis/gax-nodejs#1331)) ([d27f7fa](googleapis/gax-nodejs@d27f7fa))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
GautamSharda pushed a commit to googleapis/google-cloud-node-core that referenced this pull request Oct 29, 2025
🤖 I have created a release *beep* *boop*
---


## [3.3.1](googleapis/gax-nodejs@v3.3.0...v3.3.1) (2022-08-29)


### Bug Fixes

* **deps:** update dependency protobufjs to v7.1.0 ([#1329](googleapis/gax-nodejs#1329)) ([fc5446e](googleapis/gax-nodejs@fc5446e))
* **deps:** update dependency protobufjs-cli to v1.0.1 ([#1328](googleapis/gax-nodejs#1328)) ([b810d37](googleapis/gax-nodejs@b810d37))
* refactor createByteLengthFunction, warn if fails ([#1331](googleapis/gax-nodejs#1331)) ([d27f7fa](googleapis/gax-nodejs@d27f7fa))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
GautamSharda pushed a commit to googleapis/google-cloud-node-core that referenced this pull request Oct 29, 2025
🤖 I have created a release *beep* *boop*
---


## [3.3.1](googleapis/gax-nodejs@v3.3.0...v3.3.1) (2022-08-29)


### Bug Fixes

* **deps:** update dependency protobufjs to v7.1.0 ([#1329](googleapis/gax-nodejs#1329)) ([fc5446e](googleapis/gax-nodejs@fc5446e))
* **deps:** update dependency protobufjs-cli to v1.0.1 ([#1328](googleapis/gax-nodejs#1328)) ([b810d37](googleapis/gax-nodejs@b810d37))
* refactor createByteLengthFunction, warn if fails ([#1331](googleapis/gax-nodejs#1331)) ([d27f7fa](googleapis/gax-nodejs@d27f7fa))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants