Give the last message in the task log the correct drush message type

Looks good, although maybe the default should just be 'info'

Commited.
I also updated the default to info and satisfied the coder module.

A few remarks from the coder module:

 31 | ERROR   | Inline comments must end in full-stops, exclamation marks, or
    |         | question marks
 32 | ERROR   | Expected "switch (...) {\n"; found "switch(...){\n"
 35 | ERROR   | Case breaking statements must be followed by a single blank
    |         | line
 38 | ERROR   | Case breaking statements must be followed by a single blank
    |         | line
 41 | ERROR   | Case breaking statements must be followed by a single blank
    |         | line

While I get the reasoning here, I don't know that this is actually a good idea. FWIW, this was how I'd initially implemented this message. The problem is that we're flagging a non-warning or non-error entry as such. I think this is actually misleading.

We have jump links that get you to the first relevant warning or error. I don't know what this would provide.

Sorry for commenting after the fact.

I think it's nice that it matches the overall result of a task.

But I agree with @ergonlogic that the line itself is not an error. And it could also be confusing if someone were to gather statistics on the number of errors.

So I don't object to reverting this.

I've reverted this ... sorry @MartijnBraam