-
Notifications
You must be signed in to change notification settings - Fork 4k
Update article.md #3692
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
Update article.md #3692
Conversation
Small change in the sentence.
CLA assistant check
All committers have signed the CLA.
joaquinelio
commented
May 10, 2024
( I'm not a native English speaker)
Interesting.
I think both are correct, but it's not about grammar. They are not strictctly logically the same.
Maybe the change is easier to read, still correct?
@Afirestriker
Afirestriker
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Grammar Checked. LGTM
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
@komalagra
komalagra
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@FloffyGarlic
FloffyGarlic
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this change. The correction definitely feels more natural
smith558
commented
Jul 8, 2024
Thank you for your contribution! 🙂
Small change in the sentence.