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

Remove Many Unused Imports #3740

Merged
merged 2 commits into from
Apr 27, 2021
Merged

Remove Many Unused Imports #3740

merged 2 commits into from
Apr 27, 2021

Conversation

oke-aditya
Copy link
Contributor

@oke-aditya oke-aditya commented Apr 27, 2021

Sadly this edits a lot of files. I have touched only import lines though

All thanks to the text editor 🙏

I just opened all the files in the repo, it helped me to fix these imports 😅

@oke-aditya oke-aditya changed the title remove unused imports Remove Many Unused Imports Apr 27, 2021
Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @oke-aditya , LGTM when green (well, sort of green since CI is mostly red lately)

@datumbox datumbox merged commit 7af30ee into pytorch:master Apr 27, 2021
@oke-aditya oke-aditya deleted the rem_imp branch April 27, 2021 20:30
facebook-github-bot pushed a commit that referenced this pull request May 4, 2021
Summary: Co-authored-by: Vasilis Vryniotis <datumbox@users.noreply.github.com>

Reviewed By: NicolasHug

Differential Revision: D28169147

fbshipit-source-id: 0f1d3dce406d24a9d9b10a9c61e5fc06c5f8d9a8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants