remove labelling when cla already signed on base

[James Bruten (james-bruten-mo)]

At the moment we get a cla-signed label, even when the developer has already signed the cla on base. I'd strongly argue we should only get a cla-signed label in the PRs where the developer is signing the cla. Otherwise it'll just become meaningless noise

[James Bruten (james-bruten-mo)]

Yaswant Pradhan (@yaswant) and Andrew Coughtrie (@andrewcoughtrie) I've changed this so that I think it checks out the merged branch and then compares it to the head of the base branch. It's targeting main right now to make the diff easier to see, but I suspect we should merge into develop and test from there first. If you're happy to do that then please go ahead and change the target and merge

[Yaswant Pradhan (yaswant)]

I am not sure that will do anything different than before. But can you target develop instead of main for testing first.

[James Bruten (james-bruten-mo)]

Yaswant Pradhan (@yaswant) yeah that was my intention. I think removing the ref from the checkout action will mean it gets the PR branch merged into the base branch. And then checking that there is no diff of CONTRIBUTORS between that and the base will give us what we want. I might be wrong though!

[Andrew Coughtrie (andrewcoughtrie)]

Lets test this.