According to Drupal standards, modules should include a useful README.file.
https://www.drupal.org/docs/develop/documenting-your-project/module-docu...

Patch to follow, thanks!

CommentFileSizeAuthor
#4 location_cck-add-readme-2838115-4.patch1.95 KBddrozdik
#2 location_cck-add-readme-2838115-2.patch1.95 KBvolkswagenchick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
CreditAttribution: volkswagenchick at Hook 42 commented
Assigned: volkswagenchick » Unassigned
Status: Needs work » Needs review
FileSize
location_cck-add-readme-2838115-2.patch1.95 KB

Here is a patch that includes a README for the location_cck submodule.

thanks!

loopduplicate’s picture

loopduplicate
she/her
Location Burning Token, California
CreditAttribution: loopduplicate at Hook 42 commented
Status: Needs review » Reviewed & tested by the community

Looks good :) Marking as RTBC.

ddrozdik’s picture

ddrozdik CreditAttribution: ddrozdik as a volunteer and at FFW, Drupal Ukraine Community for FFW commented
FileSize
location_cck-add-readme-2838115-4.patch1.95 KB

Thank you for your patch. I've updated it to follow style - 80 symbols per line.

  • ddrozdik committed e9c1804 on 7.x-3.x 
    Issue #2838115 by volkswagenchick, ddrozdik, loopduplicate: README for...
ddrozdik’s picture

ddrozdik CreditAttribution: ddrozdik as a volunteer and at FFW, Drupal Ukraine Community for FFW commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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