Could it use ✅ instead of ✔️? 😂

[jozefcipa]

IMHO, it just looks cooler 🤷‍♂️ 😅

✔️ -> ✅

slack/src/slack.ts

Line 13 in e4e7168

if (status.toLowerCase() === 'success') return ':heavy_check_mark:'

[satterly]

I'm not against changing it but what if someone comes back asking for it to be changed back?

Would making it configurable work? And how would you make the config change? Using config options like this ... https://github.com/actions/checkout#usage or an environment variable? Suggestions welcome and much appreciated.

[jozefcipa]

Honestly, I don't think the icon is so important it needs a separate config option. The issue was opened without much thinking mainly out of curiosity and for fun as my colleagues and I agreed that the green icon (✅) would look better. We like the package and this is just a small thing we noticed 😅 Nevertheless if you want to update it, some optional config option should work I think 👍

[satterly]

On desktop ...

On mobile ...

[satterly]

Unfortunately the suggested check mark looks a lot worse on mobile, so 👎 😢

[satterly]

Can be configured in .github/slack.yml ...

icons:
  success: ':white_check_mark:'
  failure: ':grimacing:'
  cancelled: ':x:'
  skipped: ':heavy_minus_sign:'
  default: ':interrobang:'

Fixed by #206

[satterly]