This module is still using old README.md template format in README.md file.

Follow the pattern in the README.md template.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

Charchil Khandelwal created an issue. See original summary.

Charchil Khandelwal’s picture

Assigned: Charchil Khandelwal » Unassigned
Status: Active » Needs review

Created MR, please review.
Thank You

nupur badola’s picture

It should be ##Table of content instead of ## Contents of this file.

avpaderno’s picture

Priority: Normal » Minor
Status: Needs review » Needs work

sakthi_dev made their first commit to this issue’s fork.

sakthi_dev’s picture

Status: Needs work » Needs review

Please review.

nupur badola’s picture

Reviewed MR!2, the patch is not cleanly applied.

1. In the Table of Contents section, use - (a hyphen) instead of * (an asterisk).
2. How to use and how it works are not merged.
elber’s picture

Status: Needs review » Reviewed & tested by the community

Hi I reviewed the changes.

MR applied cleanly.

All the errors mentioned previously has been fixed.

README file is followind drupal standards.

Moving to RTBC.

avpaderno’s picture

Status: Reviewed & tested by the community » Needs work

thakurnishant_06 made their first commit to this issue’s fork.

thakurnishant_06’s picture

Status: Needs work » Needs review

Kindly review the changes.

nupur badola’s picture

Status: Needs review » Needs work

Reviewed MR!2, the patch is not cleanly applied.
1. In Supporting organization, Hyphen (-)is not required as there is only one support organization.

thakurnishant_06’s picture

Status: Needs work » Needs review
kenyoowen’s picture

Status: Needs review » Reviewed & tested by the community

Hi thakurnishant_06

I reviewed your changes and the issue mentioned previously was resolved and README.md file is following the README.md template.

Moving to RTBC
Thank you.

torfj made their first commit to this issue’s fork.

torfj’s picture

Status: Reviewed & tested by the community » Fixed

Thanks everyone! I made a few improvements and merged the changes to the dev branch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.