Skip to content

Add copyright header and pre-commit validator #1900

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

Merged
merged 12 commits into from
Mar 17, 2025
Merged

Conversation

jainapurva
Copy link
Contributor

@jainapurva jainapurva commented Mar 14, 2025

To comply with Meta Copyright Policy, added a pre-commit to add/validate it. The PR also contain files updated with the header, and some automatic pre-commit formatting fixes.

Files which contain the updated code for pre-commit:
.pre-commit-config.yaml
scripts/check_copyright_header.py

ghstack-source-id: a0e8897
Pull Request resolved: #1891
ghstack-source-id: b7b025e
Pull Request resolved: #1892
ghstack-source-id: 9dc1f9c
Pull Request resolved: #1893
ghstack-source-id: 1cca98d
Pull Request resolved: #1894
ghstack-source-id: ba10bd5
Pull Request resolved: #1895
ghstack-source-id: a43e681
Pull Request resolved: #1896
ghstack-source-id: 2010242
Pull Request resolved: #1897
ghstack-source-id: d55d4e4
Pull Request resolved: #1898
Copy link

pytorch-bot bot commented Mar 14, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/1900

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 New Failure, 2 Pending, 1 Unrelated Failure

As of commit 5e7ae13 with merge base d258a11 (image):

NEW FAILURE - The following job has failed:

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 14, 2025
@jainapurva jainapurva changed the title Add copyright header Add copyright header and pre-commit validator Mar 14, 2025
@jainapurva jainapurva added the topic: not user facing Use this tag if you don't want this PR to show up in release notes label Mar 14, 2025
@jainapurva jainapurva marked this pull request as ready for review March 14, 2025 22:07
@jainapurva jainapurva merged commit c17536e into main Mar 17, 2025
16 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. topic: not user facing Use this tag if you don't want this PR to show up in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants