This is a catch all issue to improve the documentation of drush. The patch fixes example.drushrc.php and context.inc.
| Comment | File | Size | Author |
|---|---|---|---|
| drush.docs_.patch | 2.28 KB | scor |
This is a catch all issue to improve the documentation of drush. The patch fixes example.drushrc.php and context.inc.
| Comment | File | Size | Author |
|---|---|---|---|
| drush.docs_.patch | 2.28 KB | scor |
Comments
Comment #1
scor commentedComment #2
greg.1.anderson commentedI notice a reference to 'sql load' that should be 'sql sync'. The changes made to this patch look good, though.
Comment #3
scor commentedI also found the notion of "Drupal site directory" to be confusing, that's why I added the example (e.g sites/{default|example.com}/drushrc.php). I wonder if there is a better for name for "Drupal site directory" which can easily be confused with the sites directory. maybe "Drupal multisite directory"?
Comment #4
greg.1.anderson commentedTo me, "Drupal site directory" was clear enough, but I think that your addition of the example made it much more clear. "Drupal multisite directory" to me sounds more like the "sites" directory itself (although I would call that the "Drupal sites directory"). I admit that the distinction is small enough to be confusing, but again, an example clears it up nicely.
Comment #5
moshe weitzman commentedCommitted. Feel free to reopen with new thoughts.