Skip to content

Commit

Permalink
minor #4976 Improved sentence (edsonmedina)
Browse files Browse the repository at this point in the history
This PR was submitted for the master branch but it was merged into the 2.3 branch instead (closes #4976).

Discussion
----------

Improved sentence

Commits
-------

97dcda5 Update configuration.rst
9bc95c6 Update configuration.rst
bc9500b Readability
  • Loading branch information
weaverryan committed Mar 13, 2015
2 parents a444220 + 97dcda5 commit 3be0081
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions best_practices/configuration.rst
Original file line number Diff line number Diff line change
Expand Up @@ -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 the last time was that 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:
Expand Down

0 comments on commit 3be0081

Please sign in to comment.