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

added Into<Vec<T>> #13

Merged
merged 1 commit into from
Mar 9, 2025
Merged

added Into<Vec<T>> #13

merged 1 commit into from
Mar 9, 2025

Conversation

Xgames123
Copy link

Added an impl to convert the queue into a Vec going from the most recently pushed items to the oldest ones.

@YaLTeR
Copy link
Owner

YaLTeR commented Mar 9, 2025

I updated the CI, then fixed the PR to work properly across old and new Rust.

@YaLTeR YaLTeR merged commit aa14140 into YaLTeR:master Mar 9, 2025
10 checks passed
@YaLTeR
Copy link
Owner

YaLTeR commented Mar 9, 2025

Thanks!

@YaLTeR
Copy link
Owner

YaLTeR commented Mar 9, 2025

Please give it a test, then I can publish an update

@Xgames123
Copy link
Author

Xgames123 commented Mar 9, 2025

like run the tests? (cargo test)
they all pass

@YaLTeR
Copy link
Owner

YaLTeR commented Mar 9, 2025

Like, you added this because you needed it for something?

@Xgames123
Copy link
Author

Xgames123 commented Mar 9, 2025

I trust that this will work
For my use case I did a workaround by iterating and collecting into a vec but that feels hacky because its cloning the Items

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.

2 participants