Set CLA-related labels ("CLA: Error" & "CLA: Valid")

[mgol]

Summary

For the use in jQuery, I'd like it to add a CLA: Error or CLA: Valid based on the current CLA verification status. We had this with the previous CLA system used by jQuery and it was extremely valuable - I could quickly filter PRs with a proper CLA status without opening PR links.

Background

1. Go to https://github.com/jquery/jquery/pulls
2. Be able to filter out PRs without valid CLA signatures easily.
3. Be able to tell at a glance at the PR list which PR authors have signed the CLA and which have not.

User Story

I want to be able to quickly filter PRs with a proper CLA status without opening PR links.

Acceptance Criteria

1. After EasyCLA runs on a PR, it should assign the CLA: Error or CLA: Valid depending on whether the CLA was signed or not.
2. Any time the bot re-runs the check: on PR updates, when the CLA was signed, etc., if the state of the CLA signature has changed, the previous CLA label should be removed and the proper one should be set.

References

N/A

[nickmango]

Feature to be discussed
@mlehotskylf

[mlehotskylf]

I believe the "search" functionality is a feature of GitHub, and it's not something we can address from the EasyCLA perspective. If that's the case, I suggest we close this ticket.
@nickmango @jarias-lfx please let me know if I'm wrong.

[mgol]

Search is a GitHub feature but attaching labels is very much something that the EasyCLA bot can do - and I provided rationale why that would be useful in my original post.

[nickmango]

Yea indeed @mlehotskylf we have to add the tagging accordingly on the PR

[nickmango]

@thakurveerendras kindly verify

[thakurveerendras]

Tested on dev site & found that label properly added on new & existing PR, So updating issue status

For existing PR we need to add /easycla comment

[nickmango]

Hello @mgol , thanks for your patience . Using Github Actions pipeline you can add a label upon the status of the easycla checks. Theres a project thats handling that on their end and creating labels accordingly.. This is one of the reasons it was decided that it would be more flexible with the project owner handling the status updates.
Sample script that would be of use is attached below. Kindly reach out in case of any issues you face

name: Label PR based on EasyCLA status

on:
  check_suite:
    types: [completed]
  check_run:
    types: [completed]

jobs:
  label:
    runs-on: ubuntu-latest
    steps:
    - name: Checkout code
      uses: actions/checkout@v2

    - name: Label PR
      uses: actions/github-script@v5
      if: github.event.check_suite.app.name == 'EasyCLA' || github.event.check_run.app.name == 'EasyCLA'
      with:
        script: |
          const status = (github.event.check_suite || github.event.check_run).conclusion;
          const pr_number = (github.event.check_suite || github.event.check_run).pull_requests[0].number;
          const label = status === 'success' ? 'EasyCLA:Valid' : 'EasyCLA:Error';

          await github.rest.issues.addLabels({
            owner: context.repo.owner,
            repo: context.repo.repo,
            issue_number: pr_number,
            labels: [label]
          });

cc @mlehotskylf

[mlehotskylf]

@mgol please let us know if above script worked for you. Thank you!