Skip to content

Conversation

@PaliC
Copy link
Contributor

@PaliC PaliC commented Dec 27, 2022

Stack from ghstack (oldest at bottom):

Add lintrunner support to the repo. All linters are copied over from pytorch core. Now we can just use lintrunner to do consistent linting with internal on OSS making dev a bit easier.

The PR below this #288 is just a copy and paste of the lintrunner usage in pytorch/pytorch, so this PR only has the changes specific to multipy.

The .clang-format is also taken from pytorch core.

PaliC added a commit that referenced this pull request Dec 27, 2022
ghstack-source-id: 10dec2e
Pull Request resolved: #289
PaliC added a commit that referenced this pull request Dec 27, 2022
ghstack-source-id: 77a44c8
Pull Request resolved: #289
PaliC added a commit that referenced this pull request Dec 27, 2022
ghstack-source-id: 6f81f7b
Pull Request resolved: #289
PaliC added a commit that referenced this pull request Dec 27, 2022
ghstack-source-id: c224641
Pull Request resolved: #289
PaliC added a commit that referenced this pull request Dec 27, 2022
ghstack-source-id: 502a29d
Pull Request resolved: #289
@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 4, 2023
facebook-github-bot pushed a commit that referenced this pull request Jan 11, 2023
Summary:
Pull Request resolved: #288

Here we copy over the lintrunner code from pytorch/pytorch and into multipy.
This PR is purely here to make reviewing #289 easier. This PR should be a no-op in all respects.

Test Plan: Imported from OSS

Reviewed By: kurman

Differential Revision: D42256287

Pulled By: PaliC

fbshipit-source-id: 4f161472d0fc0804f92ba73493aff6b7741b11d5
@facebook-github-bot
Copy link
Contributor

@PaliC merged this pull request in cef6cac.

@facebook-github-bot facebook-github-bot deleted the gh/PaliC/93/head branch January 15, 2023 15:18
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. Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants