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

Fixing spelling mistakes #468

Merged
merged 2 commits into from
Dec 9, 2024
Merged

Fixing spelling mistakes #468

merged 2 commits into from
Dec 9, 2024

Conversation

Danyylka
Copy link
Contributor

Description:
This pull request corrects two spelling errors in the codebase:

"amxiety" has been corrected to "anxiety."
"prevalant" has been corrected to "prevalent."
These changes ensure proper spelling and improve the overall clarity and professionalism of the project.

@Danyylka
Copy link
Contributor Author

Danyylka commented Dec 6, 2024

Hi team! Could someone please review and merge my pull request? Let me know if any changes are needed. Thanks!

@jshufro
@rex4539
@remyroy
@thorsteneb

@yorickdowne
Copy link
Collaborator

amxiety was deliberate, prevalent was a spelling error.

I am confused about these spelling PRs lately … likely AI driven, and what’s the aim?

@Danyylka
Copy link
Contributor Author

Danyylka commented Dec 8, 2024

I’m not actively creating commits and pull requests; I just noticed the error and decided to fix it. I hope this doesn’t cause any misunderstanding.

@yorickdowne
Copy link
Collaborator

Could you leave “amxiety” in place, it’s meant to be a Shiba reference. “Prevalent” does need the correction.

@Danyylka
Copy link
Contributor Author

Danyylka commented Dec 9, 2024

yes, of course I'll change that now

Copy link
Collaborator

@yorickdowne yorickdowne left a comment

Choose a reason for hiding this comment

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

Thanks

@yorickdowne yorickdowne merged commit c929e37 into eth-educators:main Dec 9, 2024
1 check passed
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