Add validation to Queues::new #137

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

Merged

brayniac merged 2 commits into main from codex/add-checks-to-queues--new-for-validity

Jun 3, 2025

Contributor

brayniac commented Jun 3, 2025

Summary

check waker lists and capacity when creating a queue
return Result from Queues::new instead of panicking
test that invalid parameters yield errors

Testing

cargo test -p switchboard --quiet
cargo test --quiet

https://chatgpt.com/codex/tasks/task_b_683f4ade19ac832aa5101655b1b838d7


          Refactor Queues::new to return Result

309369f

brayniac added the codex label

— with

ChatGPT Codex Connector


          rustfmt

108d154

brayniac merged commit fc4f9e1 into main

26 of 28 checks passed

brayniac deleted the codex/add-checks-to-queues--new-for-validity branch

June 3, 2025 22:06

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels