Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: make StreamableQuery#internalStream package-private to avoid documentation and clirr errors on change #1896

Open
wants to merge 26 commits into
base: main
Choose a base branch
from

Conversation

MarkDuckworth
Copy link
Contributor

No description provided.

MarkDuckworth and others added 23 commits September 17, 2024 11:10
…va-firestore into markduckworth/find-nearest
…va-firestore into markduckworth/find-nearest
@MarkDuckworth MarkDuckworth requested a review from a team as a code owner October 21, 2024 22:19
@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. api: firestore Issues related to the googleapis/java-firestore API. labels Oct 21, 2024
Copy link
Contributor

Choose a reason for hiding this comment

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

Could we make the entire class package private?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's possible that customers have declared variables of this type. StreamableQuery query = firestore.collection(...).where(...). I can deprecate use of this class and remove it from the API surface in the next major release.

@MarkDuckworth MarkDuckworth requested a review from a team as a code owner October 21, 2024 23:02
@product-auto-label product-auto-label bot added size: s Pull request size is small. and removed size: xs Pull request size is extra small. labels Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: firestore Issues related to the googleapis/java-firestore API. size: s Pull request size is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants