Skip to content

Conversation

@kinyoklion
Copy link
Member

@kinyoklion kinyoklion commented Feb 3, 2026

The IterableAsyncQueue was originally in common, but we've decided it doesn't need to be exposed, so I am moving it to internal. Technically this is breaking, but we have not done a release of an SDK that uses the affected version.


Note

Medium Risk
This is an API/package move that can break any downstream code importing IterableAsyncQueue, but the functional behavior change is minimal and limited to internal utility code.

Overview
Moves IterableAsyncQueue from the public com.launchdarkly.sdk.collections area into com.launchdarkly.sdk.internal.collections, making it explicitly internal rather than part of the supported API surface.

Updates the class and its test to the new package, adds package-info.java docs for the new internal package/tests, and removes the old package-info.java files from shared/common.

Written by Cursor Bugbot for commit 869e8a6. This will update automatically on new commits. Configure here.


Open with Devin

@kinyoklion kinyoklion marked this pull request as ready for review February 3, 2026 21:43
@kinyoklion kinyoklion requested a review from a team as a code owner February 3, 2026 21:43
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

@kinyoklion kinyoklion merged commit 971f4b3 into main Feb 3, 2026
22 checks passed
@kinyoklion kinyoklion deleted the rlamb/iterable-async-queue-to-internal branch February 3, 2026 21:54
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional flags.

Open in Devin Review

kinyoklion pushed a commit that referenced this pull request Feb 3, 2026
🤖 I have created a release *beep* *boop*
---


##
[1.7.0](launchdarkly-java-sdk-internal-1.6.1...launchdarkly-java-sdk-internal-1.7.0)
(2026-02-03)


### Features

* Move iterable async queue to internal.
([#125](#125))
([971f4b3](971f4b3))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Release bookkeeping only (version/changelog updates) with no
functional code changes in this diff.
> 
> **Overview**
> Prepares the `lib/shared/internal` `1.7.0` release by bumping the
version in `.release-please-manifest.json` and
`lib/shared/internal/gradle.properties`.
> 
> Updates `lib/shared/internal/CHANGELOG.md` with the `1.7.0` entry
noting the feature: moving the iterable async queue into the internal
module.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
83c9ec3. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
kinyoklion pushed a commit that referenced this pull request Feb 3, 2026
🤖 I have created a release *beep* *boop*
---


##
[2.3.0](launchdarkly-java-sdk-common-2.2.1...launchdarkly-java-sdk-common-2.3.0)
(2026-02-03)


### Features

* Move iterable async queue to internal.
([#125](#125))
([971f4b3](971f4b3))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Release bookkeeping only (version/changelog updates) with no
functional code changes in this diff.
> 
> **Overview**
> Publishes `lib/shared/common` version `2.3.0` by updating the release
manifest and the module’s `gradle.properties` version.
> 
> Updates `CHANGELOG.md` with the `2.3.0` release notes, highlighting
the feature to move the iterable async queue into the internal module.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
4d007b8. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants