This is an archive of the discontinued LLVM Phabricator instance.

[documentation]Fixed Random Typos
ClosedPublic

Authored by ipriyanshi1708 on Mar 22 2023, 10:05 AM.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptMar 22 2023, 10:05 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
ipriyanshi1708 published this revision for review.Mar 22 2023, 10:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 22 2023, 10:07 AM

Hi @ipriyanshi1708 , I am also an Outreachy applicant and I have also resolved various typos related to this issue. I will be submitting my patch very soon. We can collaborate for this issue!

Fixed one more Typo

samtebbs accepted this revision.Mar 24 2023, 4:50 AM

Thank you for this, it looks like it fixes all of the typos reported in the issue. If you don't have commit access yet I can commit this with you as the author.

This revision is now accepted and ready to land.Mar 24 2023, 4:50 AM
ipriyanshi1708 added a comment.EditedMar 24 2023, 4:53 AM

Thank you for accepting my revision. Please do commit it as I am
contributing for the first time and I don't have commit access.

samtebbs added a comment.EditedMar 27 2023, 2:13 AM

I was about to commit it for you but think that "Random" should be removed from the title, are you OK with that? Also, "Fixed" it should be in the imperative form "Fix".

Yes, I am OK with that.

This revision was automatically updated to reflect the committed changes.