[IPython-dev] Helping battle testing the newapp branch
Fernando Perez
fperez.net at gmail.com
Mon Jun 20 23:09:15 EDT 2011
On Mon, Jun 20, 2011 at 4:38 PM, MinRK <benjaminrk at gmail.com> wrote:
> Is that a 'go' for merge? If so, I'll tag&merge once I get home in an
> hour or so. Or we can wait a day or so for objections.
>
>From my side, it's a go. The ideas are sound, and we're at the point
where what we need is battle testing to shake out the problems. So I
don't see holding on having much value at this point.
Some minor comments:
- This variable seems to be ignored now, it worked in master:
c.Global.ignore_old_config = True
for people who have a single ipython_dir (like me) because they need
to handle both 0.10 and 0.11 series simultaneously, it's nice to have
a way to silence all those warnings.
- I don't think we should print the profile name in the default case,
it's just noise. I realize we now have a more consistent structure
for profiles and even the default case is now a profile, but we should
keep the amount of printed stuff to a minimum in the default cases.
Otherwise, it looks good, passes the test suite, etc.
Cheers,
f
More information about the IPython-dev
mailing list