Problem/Motivation

There are a few minor issues with the formatting in the README.md file.

Proposed resolution

Follow the pattern in the README.md template.

Issue fork views_summary_tabs-3333312

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:

About issue forks

Comments

Charchil Khandelwal created an issue. See original summary.

charchil khandelwal’s picture

charchil khandelwal commented
Assigned: Unassigned » charchil khandelwal

Charchil Khandelwal opened merge request !2

charchil khandelwal’s picture

charchil khandelwal commented
Assigned: charchil khandelwal » Unassigned
Status: Active » Needs review

Please review.

nupur badola’s picture

nupur badola commented

Review MR!2 , The patch is not cleanly applied.

git.drupalcode.org_project_views_summary_tabs_-_merge_requests_2.diff.txt:61: trailing whitespace.
Current Maintainer:
warning: 1 line adds whitespace errors.

"## Contents of this file should" change to "##Table of contents"
- (hyphen) must used in place of *

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Priority: Normal » Minor
Status: Needs review » Needs work

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

sakthi_dev’s picture

sakthi_dev commented
Status: Needs work » Needs review

Please review.

nupur badola’s picture

nupur badola commented
Status: Needs review » Needs work

Reviewed MR!2, the patch is not cleanly applied, hence moving it to Needs work. 

"1. ## Contents of this file should" change to "##Table of contents"
 2. - (hyphen) must used in place of *
nitin_lama’s picture

nitin_lama
he/him
Location India
commented
Assigned: Unassigned » nitin_lama
nitin_lama’s picture

nitin_lama
he/him
Location India
commented
Assigned: nitin_lama » Unassigned
Status: Needs work » Needs review
Akshay72’s picture

Akshay72 commented
Status: Needs review » Reviewed & tested by the community

The Readme.md File is good and looks like it has all necessary details.

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

mandclu’s picture

mandclu commented
Status: Reviewed & tested by the community » Fixed

Thanks for everyone's work on this. Merged in.

Status: Fixed » Closed (fixed)

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