Add warrior score next to its health when printing the play log about warriorjs HOT 6 CLOSED

Regular yellow is already being used, see the captive being unbound. Figured you wanted to have as many elements uniquely coloured.

Having the health bright red should be better, I'll make the PR tomorrow.

from warriorjs.

@RascalTwo I'm going to update the GIF later, but I already added a screenshot to the homepage that shows your change!

from warriorjs.

What color would you like the score line to be?

Bright Yellow looks good to me.

from warriorjs.

I like it! My only concern is that it looks like if it was more important than the health... Maybe regular yellow, or making the health bright red? BTW, feel free to open a PR at this point.

from warriorjs.

The bright red looks perfect.

I only added score to the .toJSON() method of the Warrior, since - for now at least - only the Warrior needs it's score at the moment.

I think the only documentation that needs updating due to the change is the Preview .gif. on the README.md

If you could tell me what color scheme is being used, I can update it. (I'm also a user of VSCode)

Or, another issue could be created to have the .gif updated at a later date, up to you.

from warriorjs.

Thanks @RascalTwo!

I think the only documentation that needs updating due to the change is the Preview .gif. on the README.md

Good call. Don't worry about it, I will update it later! (The color scheme is Nord, if you're still curious.)

from warriorjs.