fix: Typescript fixes

[SimpleCreations]

Overview

• Fixes issues described in Multiple issues with Typescript #118
• Defines colors as ColorValue instead of string

[mmazzarolo]

Thank you @SimpleCreations ! 🧡
I'm taking a look at it, mind fixing the linting error in the meanwhile? (Otherwise, I'll fix it later — not sure why Husky didn't fix it)

[mmazzarolo]

Code is looking solid.
Thank you for going above and beyond the TypeScript fixes!

I'll give it a quick run in a couple of hours and then I'll release it.

Wondering if this should be a major bump or not.
At a first glance, I'm not seeing cases where we're making existing types more strict. But normally, when we do these many type changes in a single shot, I prefer doing major releases to ensure we don't break existing build type-checks.

What do you think?

[mmazzarolo]

Wow, I think I haven't read this flow in ages, it's 4 years old — no idea why the strings were hardcoded in the first place, haha.
Thank you so much for cleaning this up!

[SimpleCreations]

Wondering if this should be a major bump or not.
At a first glance, I'm not seeing cases where we're making existing types more strict. But normally, when we do these many type changes in a single shot, I prefer doing major releases to ensure we don't break existing build type-checks.

What do you think?

I don't see a major version bump being necessary because yes, seems like we're only making the types less strict.

However I don't have experience maintaining public npm packages so I'm not sure about the potential pitfalls.

[github-actions]

🎉 This PR is included in version 9.0.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀