Skip to content

clean libcrux crate#1259

Open
franziskuskiefer wants to merge 13 commits intomainfrom
franziskus/main-crate
Open

clean libcrux crate#1259
franziskuskiefer wants to merge 13 commits intomainfrom
franziskus/main-crate

Conversation

@franziskuskiefer
Copy link
Member

@franziskuskiefer franziskuskiefer commented Dec 2, 2025

Draft PR for discussion of cleaned up libcrux crate.

Open todos

  • update benchmarks
  • add features
  • update CI (e.g. drop specs workflow)

cc @keks @jschneider-bensch

[skip changelog]

@jschneider-bensch
Copy link
Collaborator

I fixed the benchmarks and removed the specs workflow.

I'm unsure if we want to release this main crate soon as well, or just keep it for internal consistency and mark the released version unmaintained until e.g. we decide we want to turn it around and make the sub-crates export from the main crate.

@jschneider-bensch jschneider-bensch marked this pull request as ready for review February 24, 2026 09:24
@jschneider-bensch jschneider-bensch requested review from a team as code owners February 24, 2026 09:24
@jschneider-bensch jschneider-bensch self-requested a review February 24, 2026 09:24
@franziskuskiefer franziskuskiefer removed the request for review from jschneider-bensch February 24, 2026 09:39
Copy link
Member

@keks keks left a comment

Choose a reason for hiding this comment

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

I think this looks good, just two minor questions

@@ -0,0 +1 @@
pub use hpke_rs::*;
Copy link
Member

Choose a reason for hiding this comment

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

does that mean we don't hardcode our provider?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

is this supposed to be still here?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Are you asking if it should stay or not? I don't have it in the tree on this branch.

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

Labels

waiting-on-author Status: This is awaiting some action from the author.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants