Ah, OK. I  missed that,. That also explains why the test is failing. I'll
fix it up.


On Wed, Aug 28, 2013 at 10:34 AM, Alexei Sholik <notifications@github.com>wrote:

> In lib/iex/lib/iex.ex:
>
> > @@ -249,8 +249,11 @@ defmodule IEx do
> >    """
> >    def color(color_name, string) do
> >      colors = IEx.Options.get(:colors)
> > -    enabled = colors[:enabled]
> > -    IO.ANSI.escape_fragment("%{#{colors[color_name]}}", enabled)
> > -      <> string <> IO.ANSI.escape_fragment("%{reset}", enabled)
> > +    if colors[:enabled] do
> > +      IO.ANSI.escape_fragment("%{#{colors[color_name]}}")
> > +        <> string <> IO.ANSI.escape_fragment("%{reset}")
>
> Relying on default value for the enabled parameter is a bad idea here. It
> dynamically checks whether the output device is a terminal. In the tests,
> it is not. This is why it has to be provided in the call.
>
> Sorry that I haven't documented this in a comment.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/elixir-lang/elixir/pull/1665/files#r6036669>
> .
>