Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci] #11830

Merged
merged 1 commit into from
Sep 29, 2022

Commits on Sep 29, 2022

  1. Fix copyright check issues in pre-commit (#11711)

    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.
    
    Authors:
      - Bradley Dice (https://github.com/bdice)
    
    Approvers:
      - Jordan Jacobelli (https://github.com/Ethyling)
      - GALI PREM SAGAR (https://github.com/galipremsagar)
      - Vyas Ramasubramani (https://github.com/vyasr)
    
    URL: #11711
    bdice authored Sep 29, 2022
    Configuration menu
    Copy the full SHA
    5fad289 View commit details
    Browse the repository at this point in the history