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.

Sign up for GitHub

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

Jump to bottom

Create copy of observations in Observers.onConnected #841

Merged
merged 1 commit into from
Jan 24, 2025
Merged

Create copy of observations in Observers.onConnected #841

merged 1 commit into from
Jan 24, 2025

Conversation

twyatt
Copy link
Member

@twyatt twyatt commented Jan 20, 2025

Closes #840

@twyatt twyatt added the patch Changes that should bump the PATCH version number label Jan 20, 2025
@twyatt twyatt mentioned this pull request Jan 20, 2025
Closed
@twyatt twyatt marked this pull request as ready for review January 20, 2025 05:51
@twyatt twyatt requested a review from a team as a code owner January 20, 2025 05:51
@twyatt twyatt requested review from sdonn3 and cedrickcooke January 20, 2025 05:51
@twyatt twyatt added this to the 0.36.0 milestone Jan 20, 2025
@twyatt twyatt merged commit 8587cfe into main Jan 24, 2025
3 checks passed
@twyatt twyatt deleted the twyatt/Observers-onConnected branch January 24, 2025 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sdonn3 sdonn3 sdonn3 approved these changes

@QuantumRand QuantumRand QuantumRand approved these changes

@cedrickcooke cedrickcooke cedrickcooke approved these changes

Assignees
No one assigned
Labels
patch Changes that should bump the PATCH version number
Projects
None yet
Milestone
0.36.0
Development

Successfully merging this pull request may close these issues.

Failed to establish connection with a ConcurrentModificationException
4 participants
@twyatt @sdonn3 @QuantumRand @cedrickcooke