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 typos discovered by codespell #213

Merged
merged 2 commits into from
Feb 26, 2024
Merged

Fix typos discovered by codespell #213

merged 2 commits into from
Feb 26, 2024

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Feb 23, 2024

https://pypi.org/project/codespell

-  "description": "Boostrap renderer for FastUI",
+  "description": "Bootstrap renderer for FastUI",
                      ^

-  // fireEventImpl is recursive, but it doens't make sense for fireEvent to have fireEventImpl as a dep
+  // fireEventImpl is recursive, but it doesn't make sense for fireEvent to have fireEventImpl as a dep
                                            ^

Copy link

codecov bot commented Feb 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.68%. Comparing base (4605e6d) to head (5ce8423).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #213   +/-   ##
=======================================
  Coverage   95.68%   95.68%           
=======================================
  Files          14       14           
  Lines         950      950           
=======================================
  Hits          909      909           
  Misses         41       41           

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

@cclauss cclauss changed the title Fix typo discovered by codespell Fix typos discovered by codespell Feb 23, 2024
@hramezani
Copy link
Member

Thanks @cclauss

@hramezani hramezani merged commit c88ec12 into pydantic:main Feb 26, 2024
16 checks passed
@cclauss cclauss deleted the patch-2 branch February 26, 2024 11:05
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