As discussed in this comment thread on the original commit the NO_COLOR specification has been silently changed. The new specification expects that NO_COLOR will only be respected if set to a non-empty value.

Given the arguments in the NO_COLOR project discussion it's probably worth updating it here.

I'll submit a PR for this issue shortly.