Skip to content

Conversation

@nadzhou
Copy link

@nadzhou nadzhou commented Dec 6, 2025

Closes #xxxx

  • Tests added / passed
  • Passes pre-commit run --all-files

Closes #2007 since other classes have been implemented. Happy to get feedback on this.

@nadzhou nadzhou requested a review from fjetter as a code owner December 6, 2025 02:07
@github-actions
Copy link
Contributor

github-actions bot commented Dec 6, 2025

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

    27 files  ±  0      27 suites  ±0   10h 5m 53s ⏱️ + 5m 54s
 4 132 tests + 20   4 004 ✅  - 1    104 💤 ±0   23 ❌ + 20  1 🔥 +1 
51 781 runs  +265  49 329 ✅ +1  2 184 💤 ±0  263 ❌ +259  5 🔥 +5 

For more details on these failures and errors, see this check.

Results for commit 055fb8b. ± Comparison against base commit 5da04d0.

♻️ This comment has been updated with latest results.

@nadzhou nadzhou marked this pull request as draft December 6, 2025 06:19
Copy link
Member

@jacobtomlinson jacobtomlinson left a comment

Choose a reason for hiding this comment

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

At a quick glance this seems fine. The CI is hanging on these new tests though so you'll need to take a look at that.

Once you get things running feel free to ping for a review.

@nadzhou
Copy link
Author

nadzhou commented Dec 14, 2025

At a quick glance this seems fine. The CI is hanging on these new tests though so you'll need to take a look at that.

Once you get things running feel free to ping for a review.

Thanks for the reply. Sure let me take a look at this.

@nadzhou nadzhou force-pushed the add-condition-primitive branch from 34f5cb4 to b16e6fb Compare December 14, 2025 20:47
@nadzhou nadzhou force-pushed the add-condition-primitive branch from 470135a to 2a34eb0 Compare December 21, 2025 22:31
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.

Add Semaphore, Condition, Event

2 participants