Skip to content

reunite the workspace #1689

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

Merged
merged 10 commits into from
Apr 20, 2025
Merged

Conversation

baloo
Copy link
Member

@baloo baloo commented Feb 27, 2025

workspace got split in #1670, this bring back the crates left outside back in the workspace.

Depends:

@baloo baloo marked this pull request as draft February 27, 2025 00:08
@baloo baloo force-pushed the baloo/reunite-workspace branch from dfbc0b2 to a722417 Compare March 3, 2025 06:15
@baloo baloo force-pushed the baloo/reunite-workspace branch 2 times, most recently from 251ef95 to 030e87a Compare March 10, 2025 05:59
@baloo baloo force-pushed the baloo/reunite-workspace branch 3 times, most recently from 2142d78 to 9ae40f1 Compare March 13, 2025 02:02
@baloo
Copy link
Member Author

baloo commented Mar 13, 2025

There is something going on in the pem serialization, need to track that down. (#1711)

@baloo baloo force-pushed the baloo/reunite-workspace branch 2 times, most recently from 6ac093b to 8ed3739 Compare March 14, 2025 17:22
@baloo baloo force-pushed the baloo/reunite-workspace branch from 8ed3739 to 9f63539 Compare March 18, 2025 20:58
@tarcieri
Copy link
Member

@baloo what are the blockers here?

I worry about continuing to merge changes to der without them being properly tested against the rest of the crates.

cc @dishmaker

@baloo
Copy link
Member Author

baloo commented Apr 11, 2025

I'd need to rebase and put some time to be sure, but at that point I believe we only miss AEAD:

If I could get a merge there that would be great!

@baloo baloo force-pushed the baloo/reunite-workspace branch from 9f63539 to 60927ce Compare April 13, 2025 02:37
@baloo
Copy link
Member Author

baloo commented Apr 13, 2025

actually this will also require RustCrypto/RSA#504

@baloo baloo force-pushed the baloo/reunite-workspace branch 2 times, most recently from 6db0566 to fb3a50e Compare April 13, 2025 18:11
@baloo baloo mentioned this pull request Apr 13, 2025
1 task
@baloo baloo force-pushed the baloo/reunite-workspace branch 4 times, most recently from 76456e2 to b3353e6 Compare April 14, 2025 18:49
@baloo baloo force-pushed the baloo/reunite-workspace branch 2 times, most recently from b473244 to abb6ca0 Compare April 17, 2025 20:56
@baloo baloo force-pushed the baloo/reunite-workspace branch 2 times, most recently from 136c62f to 9f82e17 Compare April 19, 2025 16:20
@baloo baloo force-pushed the baloo/reunite-workspace branch from 9f82e17 to d5dd1bd Compare April 19, 2025 16:26
@tarcieri
Copy link
Member

@baloo can you mark this as ready if you think it's past the draft stage?

@baloo
Copy link
Member Author

baloo commented Apr 19, 2025

I need to move the git pin from my PR branch and then I can do that.

I’ll do that when I get back home (2-3h)

@baloo baloo force-pushed the baloo/reunite-workspace branch from d5dd1bd to 7d9c8b8 Compare April 19, 2025 18:44
@baloo baloo marked this pull request as ready for review April 19, 2025 18:44
@tarcieri tarcieri merged commit 58fb944 into RustCrypto:master Apr 20, 2025
166 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