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 400+ typos #1736

Closed
wants to merge 2 commits into from
Closed

Fix 400+ typos #1736

wants to merge 2 commits into from

Conversation

Lioness100
Copy link

@Lioness100 Lioness100 commented Apr 8, 2023

Let me know if there's anything I need to fix :)

fixes #1795

@CLAassistant
Copy link

CLAassistant commented Apr 8, 2023

CLA assistant check
All committers have signed the CLA.

@Lioness100 Lioness100 changed the title Fix typos Fix 400+ typos Apr 8, 2023
@paxperscientiam
Copy link

"All committers have signed the CLA." lol

@pierizvi
Copy link

pierizvi commented Apr 8, 2023

LGTM

@insomnia-creator
Copy link

LGTM, please merge.

@realKarthikNair
Copy link

LGTM

1 similar comment
@Dev-R
Copy link

Dev-R commented Apr 8, 2023

LGTM

@porterclev
Copy link

^

@saurvkmr
Copy link

saurvkmr commented Apr 9, 2023

what IDE they use at twitter? generally Intellij prompts typos!!

@wdhdev
Copy link

wdhdev commented Apr 9, 2023

This is one of the only PR's that is actually legit. LGTM!

Copy link

@pierizvi pierizvi left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@mammadyahyayev mammadyahyayev left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@fernandoxu fernandoxu left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@xhyrom xhyrom left a comment

Choose a reason for hiding this comment

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

🤔🙂 LGTM

@RaghavGoel01
Copy link

LGTM

1 similar comment
@Soulter
Copy link

Soulter commented Apr 25, 2023

LGTM

Copy link

@ravimourya ravimourya left a comment

Choose a reason for hiding this comment

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

reviewd. looks good

Copy link

@harshsingh32 harshsingh32 left a comment

Choose a reason for hiding this comment

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

looks good

harshsingh32

This comment was marked as duplicate.

Copy link

@acenoor acenoor left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link

@dromzeh dromzeh left a comment

Choose a reason for hiding this comment

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

LGTM

@Lioness100
Copy link
Author

It's been real 🫡

@Lioness100 Lioness100 closed this Jul 31, 2023
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.