-
Notifications
You must be signed in to change notification settings - Fork 15
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
CSUB-202 OCW Lock #388
CSUB-202 OCW Lock #388
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple of nitpicks, otherwise looks good!
2d43b0a
to
2c81829
Compare
This comment was marked as outdated.
This comment was marked as outdated.
2c81829
to
0a824d8
Compare
For full LLVM coverage report click here! Uncovered Lines: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Description of proposed changes:
Lock access to unverified tasks
Practical tips for PR review & merge: