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

Apply the rest of the linters in CI #6208

Merged
merged 1 commit into from
Jan 23, 2025
Merged

Apply the rest of the linters in CI #6208

merged 1 commit into from
Jan 23, 2025

Conversation

clee2000
Copy link
Contributor

@clee2000 clee2000 commented Jan 22, 2025

Added a bunch of exclusions to get it green. We should to back at some point and try to remove them
Updated grep_linter (undoes https://togithub.com/pytorch/pytorch/pull/144589)
Updated exec_linter

@clee2000 clee2000 marked this pull request as ready for review January 22, 2025 22:20
@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 Jan 22, 2025
Copy link

vercel bot commented Jan 22, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
torchci ⬜️ Ignored (Inspect) Jan 22, 2025 10:20pm

@clee2000 clee2000 requested a review from a team January 22, 2025 22:21
@clee2000 clee2000 merged commit e83685f into main Jan 23, 2025
8 checks passed
@clee2000 clee2000 deleted the csl/lint_rest branch January 23, 2025 19:15
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants