Skip to content
This repository was archived by the owner on Mar 4, 2026. It is now read-only.

fix: Tests will now verify asynchronous termination of underlying steam, and fix related bug. - #1772

Merged
tom-andersen merged 3 commits into
mainfrom
tomandersen/fixClosingOfConnection
Sep 15, 2022
Merged

tom-andersen merged 3 commits into
mainfrom
tomandersen/fixClosingOfConnection

Conversation

@tom-andersen

Copy link
Copy Markdown
Contributor

No description provided.

@conventional-commit-lint-gcf

conventional-commit-lint-gcf Bot commented Sep 14, 2022 •

Copy link
Copy Markdown

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@product-auto-label product-auto-label Bot added size: m Pull request size is medium. api: firestore Issues related to the googleapis/nodejs-firestore API. labels Sep 14, 2022
@tom-andersen
tom-andersen force-pushed the tomandersen/fixClosingOfConnection branch from 6b6e4c6 to d709194 Compare September 14, 2022 14:07
@tom-andersen
tom-andersen force-pushed the tomandersen/fixClosingOfConnection branch from d709194 to 407df2e Compare September 14, 2022 14:10
@tom-andersen tom-andersen added the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 14, 2022
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 14, 2022
@tom-andersen
tom-andersen marked this pull request as ready for review September 14, 2022 14:11
@tom-andersen
tom-andersen requested review from a team September 14, 2022 14:11
Comment thread dev/test/query.ts

@ehsannas ehsannas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Generally LGTM. Thanks for the fix.

Comment thread dev/test/query.ts
Comment thread dev/src/reference.ts
@tom-andersen
tom-andersen merged commit a1717ff into main Sep 15, 2022
@tom-andersen
tom-andersen deleted the tomandersen/fixClosingOfConnection branch September 15, 2022 18:46
gcf-merge-on-green Bot pushed a commit that referenced this pull request Sep 26, 2022
🤖 I have created a release *beep* *boop*
---


## [6.3.0](v6.2.0...v6.3.0) (2022-09-22)


### Features

* Add aggregation query APIs ([#1765](#1765)) ([18be4d0](18be4d0))


### Bug Fixes

* Tests will now verify asynchronous termination of underlying steam, and fix related bug. ([#1772](#1772)) ([a1717ff](a1717ff))

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


## [6.3.0](googleapis/nodejs-firestore@v6.2.0...v6.3.0) (2022-09-22)


### Features

* Add aggregation query APIs ([#1765](googleapis/nodejs-firestore#1765)) ([18be4d0](googleapis/nodejs-firestore@18be4d0))


### Bug Fixes

* Tests will now verify asynchronous termination of underlying steam, and fix related bug. ([#1772](googleapis/nodejs-firestore#1772)) ([a1717ff](googleapis/nodejs-firestore@a1717ff))

---
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

api: firestore Issues related to the googleapis/nodejs-firestore API. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants