Skip to content

tools: remove unused imports #5765

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed

Conversation

thefourtheye
Copy link
Contributor

Pull Request check-list

  • Does make -j8 test (UNIX) or vcbuild test nosign (Windows) pass with
    this change (including linting)?
  • Is the commit message formatted according to CONTRIBUTING.md?

Affected core subsystem(s)

tools

Description of change

This patch removes few unused imports in Python scripts

@thefourtheye thefourtheye added the tools Issues and PRs related to the tools directory. label Mar 17, 2016
@jasnell
Copy link
Member

jasnell commented Mar 17, 2016

LGTM

1 similar comment
@cjihrig
Copy link
Contributor

cjihrig commented Mar 18, 2016

LGTM

jasnell pushed a commit that referenced this pull request Mar 18, 2016
PR-URL: #5765
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@jasnell
Copy link
Member

jasnell commented Mar 18, 2016

Landed in c67937b

@jasnell jasnell closed this Mar 18, 2016
@thefourtheye thefourtheye deleted the remove-unused-imports branch March 18, 2016 18:08
Fishrock123 pushed a commit that referenced this pull request Mar 22, 2016
PR-URL: #5765
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
MylesBorins pushed a commit that referenced this pull request Mar 30, 2016
PR-URL: #5765
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
MylesBorins pushed a commit that referenced this pull request Mar 30, 2016
PR-URL: #5765
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants