Problem/Motivation

Readme.md file missing.

Proposed resolution

Create a Readme.md file.

For reference using this documentation about README.md template

CommentFileSizeAuthor
#6 Screenshot from 2023-07-03 12-48-27.png92.84 KBdineshkumarbollu

Issue fork insha-3371880

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:

  • 3379867- Comparecompare
    • 1 hidden branch
  • 3371880-readme Comparechanges, plain diff MR !9
About issue forks

Comments

Priya_Degwekar created an issue. See original summary.

priya_degwekar’s picture

priya_degwekar commented
Assigned: Unassigned » priya_degwekar

Priya_Degwekar opened merge request !9

priya_degwekar’s picture

priya_degwekar commented

Hi maintainers,
I have opened #MR9 to fix the issue.
Please review. thanks.

priya_degwekar’s picture

priya_degwekar commented
Assigned: priya_degwekar » Unassigned
Status: Active » Needs review
dineshkumarbollu’s picture

dineshkumarbollu commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Screenshot from 2023-07-03 12-48-27.png92.84 KB

Hi

I Reviewed the MR!9 looks fine According to README Template.

Thanks

priya_degwekar’s picture

priya_degwekar commented
Issue tags: +Add README.md file
akshay kashyap’s picture

akshay kashyap commented
Status: Reviewed & tested by the community » Needs work 
FILE: /home/gai/drupal-git/drupal/themes/insha/README.md
----------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
----------------------------------------------------------------------
  3 | WARNING | Line exceeds 80 characters; contains 141 characters
 22 | WARNING | Line exceeds 80 characters; contains 115 characters
 32 | WARNING | Line exceeds 80 characters; contains 83 characters
 33 | WARNING | Line exceeds 80 characters; contains 100 characters
----------------------------------------------------------------------

Please fix these warning msg

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

bandanasharma’s picture

bandanasharma commented
Status: Needs work » Needs review

I have fixed the #8 warning issued. Please review.

akshay kashyap’s picture

akshay kashyap commented
Status: Needs review » Fixed

Thanks to all the contributors. Fixed in 3.0.0 branch, included in next release.

akshay kashyap’s picture

akshay kashyap commented
Status: Fixed » Closed (fixed)

Akshay kashyap closed merge request !9