This is an archive of the discontinued LLVM Phabricator instance.

Improved Documentation Inclusivity
ClosedPublic

Authored by ps-19 on Mar 30 2022, 3:57 AM.

Details

Summary

Primary Aim Of the Patch is:

  • Added more visibility in docs.
  • Removed Errors and grammatical mistakes.
  • Removed extra spaces and tabs created to make sample code and command error-less and concise.

Dear sir/mam,
I am new to LLVM and contribution here is complicated and different. I am working on a issue, so trying to understand, how to submit a patch using this contribution. Kindly merge the request. I am really good at coding i am coding since 14 yrs age, development and open source.
https://ps-19.github.io/Resume
https://github.com/fabric8io/kubernetes-client/pull/3220
https://github.com/CleverRaven/Cataclysm-DDA/pull/51435
https://github.com/cherrypy/cherrypy/pull/1918
etc....

Diff Detail

Event Timeline

ps-19 created this revision.Mar 30 2022, 3:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 30 2022, 3:57 AM
ps-19 requested review of this revision.Mar 30 2022, 3:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 30 2022, 3:57 AM
ps-19 added a comment.Mar 30 2022, 3:58 AM

My First Contribution to LLVM please merge the patch.

ps-19 edited the summary of this revision. (Show Details)Mar 30 2022, 4:03 AM
ktkachov added inline comments.
README.md
58

I think "Most LLVM developers use Ninja" is the correct English grammar here. "Uses" would have been used for singular "developer"

ps-19 added a comment.Mar 30 2022, 4:14 AM
This comment was removed by ps-19.
ps-19 added a comment.Mar 30 2022, 4:30 AM
This comment was removed by ps-19.
ps-19 updated this revision to Diff 419103.Mar 30 2022, 4:40 AM

Removed a grammatical mistake.

ps-19 marked an inline comment as done.Mar 30 2022, 4:40 AM

Done Now!!

what should i do now? Close this patch and resend patch as a separate file again. I seriously can not understand how to edit patch?

Once you've made your changes and put them in a new patch, you can update this review with the new patch content by clicking the Update Diff link on the upper-right of this review (to the right of the Details section).

(We'll see if the picture I uploaded works or not, but it circles the button you're looking for.)

ps-19 added a comment.Mar 30 2022, 4:42 AM

what should i do now? Close this patch and resend patch as a separate file again. I seriously can not understand how to edit patch?

Once you've made your changes and put them in a new patch, you can update this review with the new patch content by clicking the Update Diff link on the upper-right of this review (to the right of the Details section).

(We'll see if the picture I uploaded works or not, but it circles the button you're looking for.)

Sir i did that!! Thank you for your kind reply. I think now build is failing.

what should i do now? Close this patch and resend patch as a separate file again. I seriously can not understand how to edit patch?

Once you've made your changes and put them in a new patch, you can update this review with the new patch content by clicking the Update Diff link on the upper-right of this review (to the right of the Details section).

(We'll see if the picture I uploaded works or not, but it circles the button you're looking for.)

Sir i did that!! Thank you for your kind reply. I think now build is failing.

It seems that you uploaded the patch directly somehow (patch_file.patch was added to this review). Btw, https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface has instructions on how to update a patch, which may help.

Sir please accept this patch i am working constantly 16 hr for consecutive 3 days to install and submit patch.

Please choose better words next time, to avoid feeling of pushing people to do something.

ps-19 updated this revision to Diff 419106.Mar 30 2022, 4:54 AM

Minor changes.

ps-19 updated this revision to Diff 419108.Mar 30 2022, 4:57 AM

Minor Changes.

ps-19 added a comment.Mar 30 2022, 5:08 AM

can anyone help me to understand why build is failing?

can anyone help me to understand why build is failing?

It looks like there are two issues:

  1. patch_file.patch is included in the patch contents when it should be the contents for the review.
  2. README.md doesn't apply cleanly: https://buildkite.com/llvm-project/diff-checks/builds/96700#9a5d304b-fa5b-4cba-a134-e16005c8ec9d

You can see the failures by clicking on the patch application failed link in Build Status to get to the link above.

ps-19 abandoned this revision.Mar 30 2022, 5:26 AM

Instead of https://reviews.llvm.org/D122727 you could just continue here with new patch revision. It would be better for you to learn things properly, instead of doing easy but incorrect ways how to solve some issues. :)

If you want learn more about phabricator, you can create new empty patch review without reviewers where you can play and learn things.

ps-19 added a comment.Mar 30 2022, 6:18 AM

Instead of https://reviews.llvm.org/D122727 you could just continue here with new patch revision. It would be better for you to learn things properly, instead of doing easy but incorrect ways how to solve some issues. :)

If you want learn more about phabricator, you can create new empty patch review without reviewers where you can play and learn things.

once my patch got merged i will surely start doing that, but right now i have to work on an issue assignment to me as soon as possible.
Thank you.

ps-19 reclaimed this revision.Mar 30 2022, 6:20 AM
ps-19 updated this revision to Diff 419125.

Last Change.

Looks fine now

ps-19 added a comment.EditedMar 30 2022, 9:22 AM

sir can you please merge it now. I am going to work on issue#53178 .

ps-19 updated this revision to Diff 419359.Mar 31 2022, 1:13 AM

Clang Formatting Checked

aaron.ballman accepted this revision.Mar 31 2022, 4:19 AM

The changes LGTM, thank you for them! What name and email address would you like me to use for patch attribution?

This revision is now accepted and ready to land.Mar 31 2022, 4:19 AM
ps-19 added a comment.Mar 31 2022, 4:30 AM

Name: Priyansh Singh
Email: priyansh.singh100@gmail.com

aaron.ballman closed this revision.Mar 31 2022, 4:41 AM

I have commit on your behalf in 1cb299165c859533e22f2ed05eb2abd5071544df -- I took the liberty of rewording the commit message a bit (FWIW, we usually pull the commit message from the patch review summary, but in this case the review summary didn't quite match what was in the patch and had a lot of extraneous information).