Skip to content

refactor: rewrite#211

Closed
glepnir wants to merge 1 commit into
mainfrom
rewrite
Closed

refactor: rewrite#211
glepnir wants to merge 1 commit into
mainfrom
rewrite

Conversation

@glepnir

@glepnir glepnir commented May 24, 2025

Copy link
Copy Markdown
Member
  1. better async handle based on async/await.
  2. Result type for error handling

@xiaoshihou514

Copy link
Copy Markdown
Member

That's a lot of code, I wouldn't be able to fully review this.

@glepnir

glepnir commented May 24, 2025

Copy link
Copy Markdown
Member Author

code is currently too messy.

@xiaoshihou514

Copy link
Copy Markdown
Member

Let's see after all tests are working again.

@glepnir

glepnir commented May 24, 2025

Copy link
Copy Markdown
Member Author

just a rough draft, still a lot of cleanup and refactoring to do.

1. better async handle based on async/await.
2. Result type for error handling
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