Skip to content

Commit

Permalink
Move high-level guidelines to main README
Browse files Browse the repository at this point in the history
  • Loading branch information
Dan Croak committed Jan 6, 2013
1 parent f977186 commit 5273cc8
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 12 deletions.
6 changes: 6 additions & 0 deletions README.md
Expand Up @@ -3,6 +3,12 @@ Guides

Guides for getting things done, programming well, and programming in style.

High level guidelines:

* Be consistent.
* Don't rewrite existing code to follow this guide.
* Don't violate the conventions without a good reason.

* [Protocol](/thoughtbot/guides/blob/master/protocol)
* [Code Review](/thoughtbot/guides/blob/master/code-review)
* [Best Practices](/thoughtbot/guides/blob/master/best-practices)
Expand Down
8 changes: 2 additions & 6 deletions code-review/README.md
@@ -1,11 +1,7 @@
Code Review
===========

A guide for respectfully reviewing code and cheerfully having your code reviewed.

Read more about the process of reviewing code, in context of writing a feature
or fixing a bug, in the [Protocol](/thoughtbot/guides/blob/master/protocol)
guide.
A guide for reviewing code and having your code reviewed.

Everyone
--------
Expand Down Expand Up @@ -73,4 +69,4 @@ An example response to style comments:

If you disagree with a style guideline, open an issue on the style guide repo
rather than discussing it within the code review. In the meantime, apply the
guideline to the codebase.
guideline.
6 changes: 0 additions & 6 deletions style/README.md
Expand Up @@ -3,12 +3,6 @@ Style

A guide for programming in style.

High level guidelines:

* Be consistent.
* Don't rewrite existing code to follow this guide.
* Don't violate the conventions without a good reason.

Git
---

Expand Down

0 comments on commit 5273cc8

Please sign in to comment.