Remove unused import
AbandonedPublic

Authored by demon on Dec 20 2017, 3:32 AM.

Details

Reviewers
mmodell
Group Reviewers
Release-Engineering-Team
Patch without arc
git checkout -b D927 && curl -L https://phabricator.wikimedia.org/D927?download=true | git apply
Summary

Found during my hunt for bad imports and such

Test Plan

Tests all ran fine

Diff Detail

demon created this revision.Dec 20 2017, 3:32 AM
Restricted Application added a reviewer: mmodell. · View Herald TranscriptDec 20 2017, 3:32 AM
Restricted Application added a reviewer: Release-Engineering-Team. · View Herald Transcript
Restricted Application added a project: Release-Engineering-Team. · View Herald Transcript
demon requested review of this revision.Dec 20 2017, 3:33 AM
mmodell requested changes to this revision.Dec 20 2017, 3:38 AM
mmodell added inline comments.
scap/arg.py
52

This isn't unused. It needs to be imported in order for the command line autocompletion to find the plugins.

This revision now requires changes to proceed.Dec 20 2017, 3:38 AM
demon added a comment.Dec 20 2017, 3:39 AM

Fair enough, but pretty much any static analysis tool will tell you otherwise. I'm slowly picking apart the dependency tree here between cli/plugins/arg :\

demon abandoned this revision.Dec 20 2017, 4:03 AM