Rename/consistency for CIs - Circle, Travis, AppVeyor about badgen.net

[amio]

• Default icon
  Better let user to choose if they want icon within. We could provide icon examples in the index page, hinting user to use it freely.

• Status consistency
  Nice to have, we need to count all possible status from all 3 CI(and more will comming), to know for sure if this consistency works flawlessly.

• Non-white icon
  I'm planning modify every non-white icon to white :D We don't provide 'social-style' badge, no need for those non-white icons.

• Text padding
  Sharp eyes! 👍 Need more tweaks on badgen 😋

[tunnckoCore]

btw, did you noticed that there is a bit more right padding than left padding in the label side,when there is icon? or it is only on phone, hm.

[tunnckoCore]

It's convention to be build | passing, but but also it's a good thing to be able to differentiate without hovering the badge to see what is the actual link behind it and which is the CI. That's why i'm for icons by default for the CIs.

[amio]

I prefer to leave status text to provider's choice, that would be best suit their definition of status. A CI could do more than just building, the 'passing' may not suit for future use cases, even it looks OK for circleci + travis + appveyor for now.

[tunnckoCore]

Sharp eyes! +1 Need more tweaks on badgen yum

It's only on mobile. Android v7 i believe, not sure.

Non-white icon

Absolute. 👍

Status consistency

I didn't get that. What you mean?

we need to count all possible status from all 3 CI

no such need, we just rename the strings and that's it, no changes to detection or any other bussiness logic.

[tunnckoCore]

Oooh, i got what you meant with 2nd bullet. Right.

[amio]

it's a good thing to be able to differentiate without hovering the badge to see what is the actual link behind it and which is the CI.

Totally agree! We could change label to CI name, keep their status as it is right now.

[tunnckoCore]

CircleCI

// :retried, :canceled, :infrastructure_fail, :timedout,
// :not_run, :running, :failed, :queued, :scheduled,
// :not_running, :no_tests, :fixed, :success
"status" : "success", 

TravisCI - damn, this guys are really bad in APIs and any other case ;d

Seems like they are few passed, failed, errored in build.state
https://docs.travis-ci.com/api/?http#builds
If there is no such repo or account, builds is empty array, so we can set unknown.

keep their status as it is right now.

Naah. Where goes the consistency ;d

AppVeyorCI

• failed, success, not sure for more, but there should be such as errored

[tunnckoCore]

keep their status as it is right now.

Or at least, if build is okey, show passing for all, otherwise their respective term for error/fail/unknown/running/quequed/timeout and etc.

[amio]

BTW, the icon color issue has been fixed in 55970a8#diff-3c6d535456832f7a3a673a4265007624R18