Let's fix INSTALL.txt and CHANGELOG.txt!

CommentFileSizeAuthor
#5 407202-we-love-all-dbs.patch1.74 KBDamien Tournoud
#3 407202-we-love-all-dbs.patch1.74 KBDamien Tournoud
#1 407202-we-love-all-dbs.patch1.73 KBDamien Tournoud
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Status: Active » Needs review
FileSize
407202-we-love-all-dbs.patch1.73 KB
keith.smith’s picture

keith.smith CreditAttribution: keith.smith commented
Status: Needs review » Needs work

The line in INSTALL.txt needs to wrap, but otherwise this should be close to RTBC.

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Status: Needs work » Needs review
FileSize
407202-we-love-all-dbs.patch1.74 KB

Rerolled, thanks Keith.

cburschka’s picture

cburschka
they
CreditAttribution: cburschka commented
Status: Needs review » Needs work

The URL for MySQL and PostGreSQL have trailing slashes, so SQLite should have one too for consistency.

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Status: Needs work » Needs review
FileSize
407202-we-love-all-dbs.patch1.74 KB

Fixed.

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Needs review » Reviewed & tested by the community
Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks!

Status: Fixed » Closed (fixed)

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