Allow unique_key to take a list

[triedandtested-dev]

resolves #2479

Description

• Support for unique_key can be a string or a list.
• merge macro support for a unique key being a list.

Checklist

• I have signed the CLA
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• I have updated the CHANGELOG.md and added information about my change

[cla-bot]

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Bryan Dunkley.
This is most likely caused by a git client misconfiguration; please make sure to:

1. check if your git client is configured with an email to sign commits git config --list | grep email
2. If not, set it up using git config --global user.email email@example.com
3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

[jtcohen6]

@triedandtested-dev Do you think you'll be able to sign the CLA? If so, I'd still love to have this change, and the corresponding changes to the default macro implementations in merge.sql as well :)

[JCZuurmond]

What is the status of this PR? I see some failed CI's and not development for two months on this

[arun-anilkumar]

What is the status of this PR? I see some failed CI's and not development for two months on this

Can we have an update please on whats holding this up?

[triedandtested-dev]

Currently I haven’t had capacity to look into this due to project commitments.

On Mon, 24 Jan 2022 at 14:53, arun-anilkumar ***@***.***> wrote: What is the status of this PR? I see some failed CI's and not development for two months on this Can we have an update please on whats holding this up? — Reply to this email directly, view it on GitHub <#4159 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AC5ZJ6L7DJMTMS7LKCZOT3LUXVRUZANCNFSM5G5VOJTQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- Bryan Dunkley

[gshank]

Bryan: Since you don't have time right now to finish this off, would you mind if we took it over? It would be nice to get it merged.

[triedandtested-dev]

Not at all. You are welcome to take it over.

On Mon, 24 Jan 2022 at 17:14, Gerda Shank ***@***.***> wrote: Bryan: Since you don't have time right now to finish this off, would you mind if we took it over? It would be nice to get it merged. — Reply to this email directly, view it on GitHub <#4159 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AC5ZJ6KETFZCTTTMWMRYDNDUXWCGZANCNFSM5G5VOJTQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- Bryan Dunkley

[gshank]

Thanks!

Note: we'd like to avoid having to update the artifact json schema for this change. See: #4617

[gshank]

This pull request has been replaced by #4618