Changed config db options from double quotes to single #1948

Merged
merged 2 commits into from Sep 21, 2015

Projects

None yet

3 participants

@laf
Member
laf commented Sep 21, 2015

I've changed our config.php.default, install.php and the docs to use single quotes so that anyone using $ in those config options won't have issues.

You can just do db_pass but there is no reason to interpret any of those config options imho.

@laf laf Changed config db options from double quotes to single
8980bbc
@laf laf Bad quotes :(
c8768cf
@Rosiak Rosiak merged commit e6306f2 into librenms:master Sep 21, 2015

2 checks passed

Auto-Deploy Build finished. No test results found.
Details
Scrutinizer No new issues
Details
@laf laf deleted the laf:config-quotes branch Jan 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment