- Issue created by @danrod
- Merge request !7Issue # 3529356: Replaced README.txt with README.md and changed a bit the formatting of the file → (Merged) created by danrod
- 🇨🇦Canada danrod Ottawa
This is ready for review, if anyone wants to take a look on it.
Hello @danrod,
I've reviewed the MR!7 as per the Readme Template → . The following points need to be addressed,- Missing two lines before ##/### headings.
- Missing a meaningful link text for links, for example: [Drupal]( https://www.drupal.org/ → ) (i.e. not just the URL)
- The introduction is added along with the project name. We can remove the duplicated introduction head and content.
- The Recommended modules head is optional, as it doesn't have any modules mentioned, so we can remove it.
- In the Installation section, if bulleted points are required, we need to use
-
instead of *. IMO, as there is no listed content, we can remove the * before the content.
Hence, moving to NW.
Thanks- 🇨🇦Canada danrod Ottawa
Thanks for reviewing this @esha_kundu , I'll look into this tomorrow early in the morning.
- 🇨🇦Canada danrod Ottawa
I applied some suggested changes, it's time for another review, but I'll probably merge it later today. Thanks @esha_kundu I'll give you the credits !
- 🇨🇦Canada danrod Ottawa
I'll merge this to the 1.0.x branch for now, @esha_kundu thanks for your input.
Automatically closed - issue fixed for 2 weeks with no activity.