Browse files

Remove commentary in style comments section

It gave conflicting advice to the "Don't violate the conventions without
a good reason." text within the style guide.
  • Loading branch information...
1 parent 2f35000 commit 2d4d1a09ccd22e6216ea776fab5b269ed7adcd1b @croaky croaky committed Jan 6, 2013
Showing with 2 additions and 5 deletions.
  1. +2 −5 code-review/README.md
View
7 code-review/README.md
@@ -70,13 +70,10 @@ guidelines. Example comment:
> Order resourceful routes alphabetically by name.
-The style guidelines are not optional. They have been agreed to as a team,
-intended to improve readability, consistency, and save time discussing style.
-
An example response to style comments:
Whoops. Good catch, thanks. Fixed in a4994ec.
-If you disagree with a style guideline, the code review is not the place to
-discuss it. Open an issue on the style guide repo. In the meantime, apply the
+If you disagree with a style guideline, open an issue on the style guide repo
+rather than discuss it within the code review. In the meantime, apply the
guideline to the codebase.

0 comments on commit 2d4d1a0

Please sign in to comment.