Skip to content

Reorganize into workspace#6

Merged
kevaundray merged 8 commits into
masterfrom
han/feature/reorg
Dec 16, 2025
Merged

Reorganize into workspace#6
kevaundray merged 8 commits into
masterfrom
han/feature/reorg

Conversation

@han0110

@han0110 han0110 commented Dec 13, 2025

Copy link
Copy Markdown
Collaborator

Inspired by #4 to reorganize the repo into workspace with 3 crates:

  • zkboost-types - Types shared between server and client
  • zkboost-client - Http client for zkboost-server
  • zkboost-server - The original impl

@kevaundray

Copy link
Copy Markdown
Contributor

Hmm strange, seems this PR was created on the weekend when we are all offline 🤔

@han0110

han0110 commented Dec 13, 2025

Copy link
Copy Markdown
Collaborator Author

no mean to disturb 🙏 just wanted to front-run #4 sooner as possible in case it'd be rebasing on my previous dirty PR (didn't notice #4 until today)

@kevaundray

Copy link
Copy Markdown
Contributor

no mean to disturb 🙏 just wanted to front-run #4 sooner as possible in case it'd be rebasing on my previous dirty PR (didn't notice #4 until today)

No worries :) Take some time off!

@kevaundray

Copy link
Copy Markdown
Contributor

I think we can close #4 and start from what we have on master, taking the good ideas from #4 -- @developeruche could you checkout this on Monday?

@kevaundray kevaundray mentioned this pull request Dec 13, 2025
@developeruche

Copy link
Copy Markdown
Contributor

I think we can close #4 and start from what we have on master, taking the good ideas from #4 -- @developeruche could you checkout this on Monday?

Sure!

@han0110 han0110 mentioned this pull request Dec 16, 2025
@han0110
han0110 marked this pull request as ready for review December 16, 2025 08:48
@kevaundray
kevaundray merged commit 21d6436 into master Dec 16, 2025
2 checks passed
@han0110
han0110 deleted the han/feature/reorg branch December 17, 2025 01:18
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