Fix copyright check issues in pre-commit

[bdice]

Description

This PR resolves #10323 and phases out the gitutils.py module in favor of a dependency on GitPython that is managed by pre-commit. It fixes the pre-commit check for copyright years so that only modifications between the target branch (branch-X.Y) and the current git stage will trigger copyright changes (years will not be updated for unmodified files, or for changes that have not been staged). Additionally, it changes the return code to 1 if changes are requested and applied (if modifications were required, that should be considered a failure).

This is the last step to making our entire style check pipeline friendly to pre-commit.

Checklist

• I am familiar with the Contributing Guidelines.
• New or existing tests cover these changes.
• The documentation is up to date with these changes.

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (branch-22.10@4681bdc). Click here to learn what that means.
Patch has no changes to coverable lines.

❗ Current head c844b55 differs from pull request most recent head 1f9d28f. Consider uploading reports for the commit 1f9d28f to get more accurate results

Additional details and impacted files

@@               Coverage Diff               @@
##             branch-22.10   #11711   +/-   ##
===============================================
  Coverage                ?   85.90%           
===============================================
  Files                   ?      151           
  Lines                   ?    23532           
  Branches                ?        0           
===============================================
  Hits                    ?    20214           
  Misses                  ?     3318           
  Partials                ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[bdice]

@vyasr @galipremsagar I tagged you both for review because of your experience with the copyright scripts and pre-commit. Thanks!

[galipremsagar]

Once merged, could you also copy over the changes to cuml: https://github.com/rapidsai/cuml/blob/branch-22.10/ci/checks/copyright.py, These scripts have been copied over from there actually.

[ttnghia]

Is this good to go?

[bdice]

Is this good to go?

I’d like a final approval from @vyasr to make sure there wasn’t anything else he wanted to address. Then please feel free to merge!

[vyasr]

I couldn't come up with a nice way to resolve the file vs git.Diff object issue, I think we just need to support both for now. Aside from that, this looks fine.

[vyasr]

@gpucibot merge