#84 Update modules/ROOT/pages/mentored_project/Info.adoc
Merged by siddharthvipul1. Opened by t0xic0der.
t0xic0der/mentored-projects master  into  master

Download 84.patch

Corrected a few typos, capitalizations etc.

Looks good to me. I am +1 and good to merge
@siddharthvipul1 since you are the mentored project rep, just put a +1 / if you don't like anything, let's discuss

siddharthvipul1 commented

@t0xic0der can you wrap lines on 80 characters? They are easier to review and that's the way it should have been earlier as well.
Will merge once you also amended that ;)
If that seems like too much work, I can fix it in another commit as well.
just let me know

1 new commit added

  • Update modules/ROOT/pages/mentored_project/Info.adoc

Brought down the per-line character count to less than or equal to 80. Please be careful about the possible broken links though which might have crept in during this formatting.

siddharthvipul1 commented

built locally, all looks great! Thank you @t0xic0der for fixing these :)

Pull-Request has been merged by siddharthvipul1

siddharthvipul1 commented

ah, I missed one link not rendering (funny enough it was not rendered earlier as well)
I will fix it on Monday evening

@t0xic0der can you wrap lines on 80 characters? They are easier to review and that's the way it should have been earlier as well.

Hi @siddharthvipul1 @t0xic0der! Quick note, for AsciiDoc documentation, the convention is to wrap each sentence on its own line. This makes git diffs easier to review for content and also encourages a habit of writing shorter, easier-to-understand sentences.

https://docs.fedoraproject.org/en-US/fedora-docs/asciidoc-fedora/markup/#basics

https://asciidoctor.org/docs/asciidoc-recommended-practices/#one-sentence-per-line

Please try to follow this convention in Fedora Docs so it is easier for other editors in the Docs community to work on. Thanks! :grin:

Metadata