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

Fix Vec pre-alloc panic #136

Merged
merged 6 commits into from
Apr 3, 2024
Merged

Conversation

isikkema
Copy link
Contributor

@isikkema isikkema commented Apr 2, 2024

Fixes and implements the changes suggested in #135.

Copy link

netlify bot commented Apr 2, 2024

Deploy Preview for cute-starship-2d9c9b canceled.

Name Link
🔨 Latest commit 0191bd6
🔍 Latest deploy log https://app.netlify.com/sites/cute-starship-2d9c9b/deploys/660d78f519e7fb0008b21aeb

Copy link
Owner

@jamesmunns jamesmunns left a comment

Choose a reason for hiding this comment

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

This looks good to me, happy to merge. I'm unsure if this will be released until a potential future 2.0 release.

@jamesmunns jamesmunns merged commit 6338601 into jamesmunns:main Apr 3, 2024
4 checks passed
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