We could be using README.md instead of README.txt to make use of the nicer rendering via markdown.

Now that we've moved to GitLab, we should probably use Markdown instead of plain .txt files for our documentation

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

Nila Hyalij created an issue. See original summary.

Nila Hyalij’s picture

Status: Active » Needs review

Committed changes to proposed solution.
Please review it.

Pracheth’s picture

Assigned: Nila Hyalij » Unassigned
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new84.76 KB

Reviewed the README.md (latest MR! by Nila Hyalij#2) , The Markdown looks good to me adhering to Drupal markdown practices.

Changing status to Reviewed and Tested by Community.

tyler36’s picture

@Pracheth

Where are the "Drupal markdown practises"?

I found Markdown
which contains a link to a proposal and a postponed policy.

ressa’s picture

Issue summary: View changes
Status: Reviewed & tested by the community » Needs work

Please, see the README.md template.

shubham rathore’s picture

Assigned: Unassigned » shubham rathore
shubham rathore’s picture

Assigned: shubham rathore » Unassigned
Status: Needs work » Needs review

Update README.md according to README.md template.
Please review.
Thanks

nupur badola’s picture

Status: Needs review » Reviewed & tested by the community

Patch is working and marking it to RTBC.

Manoj Raj.R’s picture

Status: Reviewed & tested by the community » Needs work
Manoj Raj.R’s picture

Updated the changes with some minor fixes.

Please review.

Manoj Raj.R’s picture

Status: Needs work » Needs review
kavitha89’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new15.27 KB

The patch works fine. Moving it to RTBC

avpaderno’s picture

Priority: Normal » Minor
Status: Reviewed & tested by the community » Needs work
nitin_lama’s picture

Assigned: Unassigned » nitin_lama
nitin_lama’s picture

Assigned: nitin_lama » Unassigned
Status: Needs work » Needs review
Bushra Shaikh’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new75.71 KB
new68.06 KB

I verified and applied MR !3 for Drupal version 10.0.2.
README.txt file updated to README.md after applying this patch.

can be move to RTBC+1

avpaderno’s picture

Status: Reviewed & tested by the community » Needs work
nitin_lama’s picture

Assigned: Unassigned » nitin_lama
nitin_lama’s picture

Status: Needs work » Needs review

Please review.

nitin_lama’s picture

Assigned: nitin_lama » Unassigned
nupur badola’s picture

Unable to understand : The subject is missing and the verb is declined on the first person singular, or the subject is implicit and the verb is using the Imperative Mood. In either the cases, the sentence is not correct.

kenyoowen’s picture

Status: Needs review » Needs work

Hi nitin_lama

The thread for the introduction and project name in comment #18 is not resolved. This is the introduction and project name issue Remove "Request new password" link from block and user page.

Thank you

avpaderno’s picture

SInce that section is describing the module, Remove "Request new password" link from block and user page. (which would use the imperative mood) is not correct.
The module removes the "Request new password" link from the block and the user page. is correct.

nitin_lama’s picture

Assigned: Unassigned » nitin_lama
nitin_lama’s picture

Status: Needs work » Needs review

Please review.

nitin_lama’s picture

Assigned: nitin_lama » Unassigned
avpaderno’s picture

Status: Needs review » Needs work
nitin_lama’s picture

Assigned: Unassigned » nitin_lama
nitin_lama’s picture

Status: Needs work » Needs review

Please review.

nitin_lama’s picture

Assigned: nitin_lama » Unassigned
Bushra Shaikh’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new73.01 KB

I verified and applied MR !3, README.txt file updated to README.md after applying this patch.
Adding screenshots for reference.

Can be move to RTBC+1

Neslee Canil Pinto made their first commit to this issue’s fork.

neslee canil pinto’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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