Closed (fixed)
Project:
Stage File Proxy
Version:
8.x-1.x-dev
Component:
Documentation
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
3 Feb 2017 at 19:34 UTC
Updated:
13 May 2017 at 08:34 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
volkswagenchickHere is a useful README for the d8 branch. Thanks for this VERY USEFUL module.
Comment #3
loopduplicate commentedLooks good except for:
special character at the end of the sentence needs to be removed.
Special character and extra space at the end of this line should be removed.
Regards,
Jeff
Comment #4
volkswagenchickThanks Loop. Here is a new patch, along with an interdiff, with the extra spaces removed. THANKS again!
Comment #5
loopduplicate commentedLooks good :)
Comment #6
mark_fullmerDrupal 8 doesn't support variable_set and $conf variables as described here. Apologies if I'm missing something, but this Readme seems applicable to 7.x, not 8.x.
I might encourage the OP to do a bit more diligence when creating READMEs. Quality over quantity! https://www.drupal.org/user/3332522/track/code
Comment #7
mark_fullmerThe 8.x configuration is already thoroughly documented by the maintainers in the existing INSTALL.txt file. So, per module documentation guidelines, I feel it can remain there and the README can provide the "basic overview of what the module does and how someone may use it".
The patch attached does that, and moves the INSTALL.txt file to a INSTALL.md file.
Comment #8
loopduplicate commentedSounds good to me :) I wonder what you all think about wrapping consistency? Here's my thoughts:
Some lines break at 80 characters and some don't. Probably best to keep it consistent. Since it's not code, it's ok to extend beyond the 80 character mark but it seems fine to restrict all lines too.
Same as above, this line goes beyond the 80 char mark whereas other lines are wrapped at 80 chars.
When wrapping text, there shouldn't be a space before the wrapped lines.
Regards,
Jeff
Comment #9
BarisW commentedThanks all! I fixed the wrapping issues when committing it.
Comment #11
BarisW commented