Skip to content

Migrate from yanked core2 library to corez#482

Merged
nuttycom merged 2 commits into
mainfrom
corez_migration
Apr 17, 2026
Merged

Migrate from yanked core2 library to corez#482
nuttycom merged 2 commits into
mainfrom
corez_migration

Conversation

@nuttycom
Copy link
Copy Markdown
Contributor

No description provided.

nullcopy
nullcopy previously approved these changes Apr 17, 2026
Copy link
Copy Markdown

@nullcopy nullcopy left a comment

Choose a reason for hiding this comment

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

utACK e94ade8 modulo rustfmt errors

@nuttycom nuttycom linked an issue Apr 17, 2026 that may be closed by this pull request
Copy link
Copy Markdown

@nullcopy nullcopy left a comment

Choose a reason for hiding this comment

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

utACK 872a219

Copy link
Copy Markdown
Contributor

@daira daira left a comment

Choose a reason for hiding this comment

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

utACK

@nuttycom nuttycom merged commit b0bf267 into main Apr 17, 2026
32 checks passed
@nuttycom nuttycom deleted the corez_migration branch April 17, 2026 20:06
Comment thread Cargo.toml

# No-std support
core2 = { version = "0.3", default-features = false, features = ["alloc"] }
corez = { version = "0.1.1", default-features = false, features = ["alloc"] }
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I checked to see if corez 0.1.1 was a 1:1 match of the yanked core2 0.3, but it seems to be a slimmed down version. I checked the diff and the things that were removed appear safe to me.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Correct; in particular, some code paths that could result in UB were removed; this is the minimal API surface that our crates actually need.

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.

Migrate from yanked core2 crate to corez

4 participants