[IPython-dev] Style Rules for Notebook
Thomas Kluyver
takowl at gmail.com
Wed Aug 22 13:09:16 EDT 2012
On 22 August 2012 17:42, Matthias BUSSONNIER
<bussonniermatthias at gmail.com> wrote:
> Don't hesitate to open a PR even before finishing the code
Just to expand on this: if you're making a PR for discussion, and you
know the code isn't finished, you can start the description with a
note like "not ready for merge".
If you've done everything you want to do with the code, no disclaimer
is necessary, but even so, don't be surprised if we ask you to make a
few changes. Almost every PR (apart from typo fixing) gets some
changes before it's merged. Code review is really important, even for
core developers.
Best wishes,
Thomas
More information about the IPython-dev
mailing list