Skip to content

fix: replaced empty tuple error types with Infallible#59

Merged
tarcieri merged 2 commits intoRustCrypto:masterfrom
AlexSherbinin:fixes
Oct 19, 2024
Merged

fix: replaced empty tuple error types with Infallible#59
tarcieri merged 2 commits intoRustCrypto:masterfrom
AlexSherbinin:fixes

Conversation

@AlexSherbinin
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread dhkem/Cargo.toml Outdated
@tarcieri
Copy link
Copy Markdown
Member

tarcieri commented Sep 5, 2024

Seems like a breaking change, but we'll likely need breaking changes to address #53 as well

@faern
Copy link
Copy Markdown

faern commented Oct 3, 2024

Yes please! This would be great. Any news on getting this merged?

@tarcieri
Copy link
Copy Markdown
Member

Looks like you'll need to update how the errors are handled in x-wing, since this is a breaking change

@tarcieri
Copy link
Copy Markdown
Member

If you rebase it should now run the full x-wing test suite when ml-kem is changed

@tarcieri tarcieri merged commit 502ff6b into RustCrypto:master Oct 19, 2024
tarcieri added a commit that referenced this pull request Oct 21, 2024
There were breaking changes in #59.

This denotes that a breaking change occurred, however there won't be a
release associated with this commit. Instead we'll release a `pre.0`
or a v0.3.0 final.
tarcieri added a commit that referenced this pull request Oct 21, 2024
There were breaking changes in #59.

This denotes that a breaking change occurred, however there won't be a
release associated with this commit. Instead we'll release a `pre.0`
or a v0.3.0 final.
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.

3 participants