-
Notifications
You must be signed in to change notification settings - Fork 194
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
chore: accurate code comments and minor code changes #649
chore: accurate code comments and minor code changes #649
Conversation
@aFlyBird0 Please check the ci error. |
Sorry, new code has been committed. (I ran |
Very detailed change by @aFlyBird0. Thanks very much. I committed a few grammar updates in the PR directly; hope you won't mind. Please do a regression test (for example, the GitOps process) to make sure the code change didn't break anything. |
@aFlyBird0 keep up the good work! |
Thanks for your additional commits sincerely! I will do the regression test this afternoon. @IronCore864 |
@aFlyBird0 @IronCore864 How about merge this pr now. I'll test it locally today. I'm also changing the logic of the package |
Pre-Checklist
Note: please complete ALL items in the following checklist.
Description