Skip to content
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

fix:some typos #30

Merged
merged 1 commit into from
Mar 18, 2024
Merged

fix:some typos #30

merged 1 commit into from
Mar 18, 2024

Conversation

yuanbohan
Copy link
Collaborator

What's changed and what's your intention?

fix some typos

Checklist

  • I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.53%. Comparing base (99663b9) to head (ce8aa70).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #30   +/-   ##
=======================================
  Coverage   76.53%   76.53%           
=======================================
  Files           7        7           
  Lines         439      439           
=======================================
  Hits          336      336           
  Misses         80       80           
  Partials       23       23           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

@fengjiachun fengjiachun left a comment

Choose a reason for hiding this comment

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

LGTM

@yuanbohan
Copy link
Collaborator Author

@JetSquirrel PTAL

@JetSquirrel
Copy link
Contributor

@yuanbohan Thanks for the fixes, I'll keep an eye out for them afterwards.

@yuanbohan yuanbohan merged commit 17a025d into main Mar 18, 2024
5 checks passed
@yuanbohan yuanbohan deleted the fix-typo branch March 18, 2024 04:26
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.

None yet

4 participants