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

Have computed var properties also use the queue to add invocations safely #334

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

welshm
Copy link

@welshm welshm commented Dec 20, 2022

I believe this fixes #333

@welshm
Copy link
Author

welshm commented Dec 20, 2022

@welshm
Copy link
Author

welshm commented Dec 20, 2022

@navartis @spaluchiewicz looks like you're the most recent committers? Are you reviewing pull requests for this repo?

Copy link

@maxim-chipeev maxim-chipeev left a comment

Choose a reason for hiding this comment

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

Awesome, we need this for testing in an async environment

@welshm
Copy link
Author

welshm commented Jan 5, 2023

Anything I can do to help get this reviewed and merged?

@maxim-chipeev
Copy link

@g-amichnia Hi, I saw you were the second most recent reviewer…would you be so kind as to take a look?

@cassianomonteiro
Copy link

+1 on this.

@cassianomonteiro
Copy link

@spaluchiewicz Would you be able to take a look at this?

@roelspruit
Copy link

Hi. we ran into this issue in our project. Is somebody still going to review this MR? We'd love to have a fix for this issue in an official release.

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.

Computed vars do not write to invocation array in a threadsafe way
4 participants