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 unused imports #35

Closed
wants to merge 1 commit into from
Closed

Remove unused imports #35

wants to merge 1 commit into from

Conversation

marschall
Copy link
Contributor

Several classes contain imports that are not unused. They can be
removed without effect.

Several classes contain imports that are not unused. They can be
removed without effect.
@ajlam ajlam added the Under Review Used for pull requests under review label Nov 28, 2016
@xiangyushawn xiangyushawn removed the Under Review Used for pull requests under review label Nov 30, 2016
@xiangyushawn
Copy link
Contributor

@marschall thank you for your contribution. Our team is currently working on removing unused imports and wildcards and will be checked in very soon.

@marschall
Copy link
Contributor Author

@v-xiangs that is great to hear, I'm looking forward to it. Will you also be fixing the indentation (currently it's a mix between spaces and tabs), remove trailing spaces and use a consistent method style (currently it's a mix between braces on the same line and braces on the next line) or should I file separate issues for these?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants