Problem/Motivation

Replace README.txt with README.md

Steps to reproduce

update the README.md formate

Proposed resolution

following this pattern
https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or...

CommentFileSizeAuthor
#2 3315929.patch1.13 KBanchal_gupta

Issue fork migrate_tools-3315929

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

dineshreddymathuku created an issue. See original summary.

anchal_gupta’s picture

anchal_gupta commented
StatusFileSize
new 3315929.patch1.13 KB

I have uploaded the patch. Please review it

Manoj Raj.R made their first commit to this issue’s fork.

Manoj Raj.R opened merge request !26

Manoj Raj.R’s picture

Manoj Raj.R commented
Status: Active » Needs review
dineshreddymathuku’s picture

dineshreddymathuku commented

Hi #2 and #3 looks good for me
thanks for your Response

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Needs work

The template is for a plain text file, it doesn't fit the format or structure needed for a Markdown file. This needs more work to fit the Markdown syntax better.

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

gowthamrajkrishnasamy opened merge request !27

gowthamrajkrishnasamy’s picture

gowthamrajkrishnasamy commented
Status: Needs work » Needs review

Commited as per #7

dineshreddymathuku’s picture

dineshreddymathuku commented

Updated as per recomended markdown format please review it once

nupur badola’s picture

nupur badola commented

Issue exists in the patch.

git.drupalcode.org_project_migrate_tools_-_merge_requests_27.diff.txt:9: trailing whitespace.
 * Introduction
warning: src/MigrateExecutable.php has type 100644, expected 100755
error: patch failed: src/MigrateExecutable.php:338
error: src/MigrateExecutable.php: patch does not apply
nupur badola’s picture

nupur badola commented
Status: Needs review » Needs work

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

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Needs work » Reviewed & tested by the community

No commit credit granted for converting the readme to a markdown since it is mainly just a copy/paste. Thanks for your contributions.

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Reviewed & tested by the community » Fixed
heddn’s picture

heddn
Primary language English
Location Illinois
commented
Version: 8.x-5.x-dev » 6.0.x-dev

Status: Fixed » Closed (fixed)

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

heddn closed merge request !26