While evaluating this module, I noticed it lacked a readme.
According to Drupal standards, modules should include a useful README.file.
https://www.drupal.org/docs/develop/documenting-your-project/module-docu....

https://www.drupal.org/docs/develop/documenting-your-project/readme-temp...

Patch to follow, thanks!

CommentFileSizeAuthor
#2 ctools-d7-fix-readme-2987162-2.patch3.42 KBvolkswagenchick

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Assigned: volkswagenchick » Unassigned
Status: Active » Needs work
StatusFileSize
new ctools-d7-fix-readme-2987162-2.patch3.42 KB

I uploaded a patch that adds a readme file. The readme has some basic info about the modules, useful links (project page, issue queue, installation), and is formatted to align with Drupal documentation standards.

Thanks for all the work on this very powerful module.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Status: Needs work » Needs review
dani3lr0se’s picture

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

The patch applies cleanly. Formatting, spelling, grammar and readability all look good. Makes sense and easy to follow. It aligns with Drupal standards as well. Thanks for the patch @volkswagenchick. :)

rivimey’s picture

rivimey
she/her
commented
Parent issue: » #3031732: Plan for CTools 7.x-1.16 release

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Reviewed & tested by the community » Fixed
Parent issue: #3031732: Plan for CTools 7.x-1.16 release » #3178513: Plan for CTools 7.x-1.17 release

@volkswagenchick I've removed the Maintainers section on commit because it's so fluid over time, but have committed it to the dev branch. Thanks for your help and patience on this.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented

makes sense to remove the maintainers. Thanks!

Status: Fixed » Closed (fixed)

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