janestreet/iron

Jane Street code review system

189

stars

56

commits

Jan 14, 2020

updated

Browse cluster: OCaml metaprogramming and PPX

README

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/

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/

We no longer release the code of Iron, and do not expect that will change.
It was never possible to run outside of Jane Street, and is not plausible
that it ever will, due to dependencies on many aspects of our internal
environment.

Not written in Markdown, so it's shown here as plain text — view it formatted on GitHub.

Contributors

jdimino-js

41 commits

trefis

12 commits

jeremiedimino

1 commits

jsoref

1 commits

janestreet/iron

Jane Street code review system

189

stars

56

commits

Jan 14, 2020

updated

Browse cluster: OCaml metaprogramming and PPX

README

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/

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/

We no longer release the code of Iron, and do not expect that will change.
It was never possible to run outside of Jane Street, and is not plausible
that it ever will, due to dependencies on many aspects of our internal
environment.

Not written in Markdown, so it's shown here as plain text — view it formatted on GitHub.

Contributors

jdimino-js

41 commits

trefis

12 commits

jeremiedimino

1 commits

jsoref

1 commits