From bc9500bb46a1c7b6a6219ff5a126d93ca2c76e6a Mon Sep 17 00:00:00 2001 From: Edson Medina Date: Sun, 8 Feb 2015 14:11:31 +0000 Subject: [PATCH 1/3] Readability --- best_practices/configuration.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/best_practices/configuration.rst b/best_practices/configuration.rst index 649c97a197a..4135f30aeea 100644 --- a/best_practices/configuration.rst +++ b/best_practices/configuration.rst @@ -103,7 +103,7 @@ to control the number of posts to display on the blog homepage: parameters: homepage.num_items: 10 -If you ask yourself when the last time was that you changed the value of +If you ask yourself when was the last time you changed the value of *any* option like this, odds are that you *never* have. Creating a configuration option for a value that you are never going to configure just isn't necessary. Our recommendation is to define these values as constants in your application. From 9bc95c676897b94c4c12edd9e30e1c7a376d3a2b Mon Sep 17 00:00:00 2001 From: Edson Medina Date: Sun, 8 Mar 2015 13:54:00 +0000 Subject: [PATCH 2/3] Update configuration.rst --- best_practices/configuration.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/best_practices/configuration.rst b/best_practices/configuration.rst index 4135f30aeea..db0d73c45f6 100644 --- a/best_practices/configuration.rst +++ b/best_practices/configuration.rst @@ -103,8 +103,8 @@ to control the number of posts to display on the blog homepage: parameters: homepage.num_items: 10 -If you ask yourself when was the last time you changed the value of -*any* option like this, odds are that you *never* have. Creating a configuration +If you've done something like this in the past, it's likely that you've in fact *never* actually needed +to change that value. Creating a configuration option for a value that you are never going to configure just isn't necessary. Our recommendation is to define these values as constants in your application. You could, for example, define a ``NUM_ITEMS`` constant in the ``Post`` entity: From 97dcda53932b4c1633cdd03828bd71e11695c43c Mon Sep 17 00:00:00 2001 From: Edson Medina Date: Sun, 8 Mar 2015 17:38:57 +0000 Subject: [PATCH 3/3] Update configuration.rst --- best_practices/configuration.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/best_practices/configuration.rst b/best_practices/configuration.rst index db0d73c45f6..ff1d8d05428 100644 --- a/best_practices/configuration.rst +++ b/best_practices/configuration.rst @@ -103,8 +103,8 @@ to control the number of posts to display on the blog homepage: parameters: homepage.num_items: 10 -If you've done something like this in the past, it's likely that you've in fact *never* actually needed -to change that value. Creating a configuration +If you've done something like this in the past, it's likely that you've in fact +*never* actually needed to change that value. Creating a configuration option for a value that you are never going to configure just isn't necessary. Our recommendation is to define these values as constants in your application. You could, for example, define a ``NUM_ITEMS`` constant in the ``Post`` entity: