Closed (outdated)
Project:
Drupal core
Version:
7.x-dev
Component:
base system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
29 Dec 2009 at 06:31 UTC
Updated:
29 Mar 2015 at 04:08 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
info@zrix.com commentedComment #3
aspilicious commentedI'll try to help things going...
A try to fix previous patch
1) remove windows line endings (check)
2) 2 spaces indentation (check)
3) no tabs (check)
4) readable patch name... (check)
5) path name (check)
6) test on local machine (check)
Ok lets see what bot thinks
Comment #4
aspilicious commentedComment #5
aspilicious commentedOk it passed...
Now the real reviews can come in
Comment #6
MichaelCole commented#3: action-add-static-cache-669732-3.patch queued for re-testing.
Comment #7
cwgordon7 commentedThe current code reads:
this could be shortened to:
I think that the second is more concise and more readable.
Comment #8
silverwing commentedComment #9
mgiffordUnassigning.