Skip to content

Commit

Permalink
layout
Browse files Browse the repository at this point in the history
  • Loading branch information
trefis committed Apr 24, 2018
1 parent 229efa1 commit af43b7c
Showing 1 changed file with 7 additions and 8 deletions.
15 changes: 7 additions & 8 deletions README.org
Original file line number Diff line number Diff line change
Expand Up @@ -2,18 +2,17 @@ Iron is Jane Street's internal code review tool.

For more on the design of iron, see the following blog posts:

https://blog.janestreet.com/code-review-that-isnt-boring/
https://blog.janestreet.com/scrutinizing-your-code-in-style/
https://blog.janestreet.com/ironing-out-your-release-process/
https://blog.janestreet.com/putting-the-i-back-in-ide-towards-a-github-explorer/
- https://blog.janestreet.com/code-review-that-isnt-boring/
- https://blog.janestreet.com/scrutinizing-your-code-in-style/
- https://blog.janestreet.com/ironing-out-your-release-process/
- https://blog.janestreet.com/putting-the-i-back-in-ide-towards-a-github-explorer/

A key design decision was to review diffs rather than patches:

https://blog.janestreet.com/designing-a-code-review-tool-part-2-patches-or-diffs/
https://blog.janestreet.com/patch-review-vs-diff-review-revisited/
- https://blog.janestreet.com/designing-a-code-review-tool-part-2-patches-or-diffs/
- https://blog.janestreet.com/patch-review-vs-diff-review-revisited/

We no longer release the code of iron, not because the code is proprietary
but
because one couldn't run the code elsewhere without first reproducing many
but because one couldn't run the code elsewhere without first reproducing many
trivial features of our internal environment. We're still happy to share or
discuss the code, though: just email [email protected].

0 comments on commit af43b7c

Please sign in to comment.