Update `VariantProp` type merging logic for `Compose` interface

lunelson commented 6 months ago

Description

This PR primarily addresses the following issue, updating the variant prop merging logic that is applied by the Compose interface, so that the union of available variant prop names is correct.

https://github.com/joe-bell/cva/issues/256

There are also some secondary simplifications of types, and application of unknown in place of any.

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Follow the Style Guide.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).

vercel[bot] commented 6 months ago

@lunelson is attempting to deploy a commit to the cva Team on Vercel.

A member of the Team first needs to authorize it.

joe-bell commented 6 months ago

Thanks @lunelson! Bit stretched for time at the moment but I'll try to get around to testing this. Really appreciate the contribution 🙏

kaelansmith commented 2 weeks ago

@lunelson glad to see this getting pushed forward, thanks! @joe-bell are you still too busy to push this over the finish line? Any way I can help?

joe-bell / cva