While evaluating this module, while it has a readme, I noticed the readme could use some helpful links and configurations and be reformatted to alig to Drupal documentation standards.
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...

FILE: /home/thallesvf/Desktop/lando/d8-contrib/modules/imce/README.txt
----------------------------------------------------------------------
FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 5 LINES
----------------------------------------------------------------------
 23 | WARNING | Line exceeds 80 characters; contains 81 characters
 24 | WARNING | Line exceeds 80 characters; contains 96 characters
 35 | WARNING | Line exceeds 80 characters; contains 99 characters
 37 | WARNING | Line exceeds 80 characters; contains 102 characters
 38 | WARNING | Line exceeds 80 characters; contains 95 characters
----------------------------------------------------------------------

Patch to follow, thanks!

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

Assigned: volkswagenchick » Unassigned
Status: Active » Needs review
StatusFileSize
new3.26 KB

I uploaded a patch that better aligns the readme to Drupal documentation standards, thanks!

alonaoneill’s picture

Status: Needs review » Reviewed & tested by the community

Reviewed the patch for spelling and grammar. Formatting aligns to Drupal documentation standards. Everything looks good. Patch applied.
Marking as RTBCed.
Thank you for working on module!

thalles’s picture

Can you show here the output of phpcs, before and after the changes?

thalles’s picture

Status: Reviewed & tested by the community » Postponed (maintainer needs more info)
thalles’s picture

Issue summary: View changes
thalles’s picture

Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new3.27 KB
new431 bytes

Follow the patch!

thalles’s picture

StatusFileSize
new3.27 KB
new567 bytes

Follow a new patch!

  • thalles committed 371c1c3 on 8.x-1.x
    Issue #3014274 by thalles, volkswagenchick, alonaoneill: README configs...
thalles’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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