Couple of minor cleanups to setup.py, based on runs from Pyroma
ClosedPublic

Authored by demon on Dec 6 2017, 10:52 PM.

Details

Reviewers
mmodell
Group Reviewers
Release-Engineering-Team
Commits
rMSCA7244e95ada2a: Couple of minor cleanups to setup.py, based on runs from Pyroma
Patch without arc
git checkout -b D909 && curl -L https://phabricator.wikimedia.org/D909?download=true | git apply
Summary
  • Add some keywords to describe scap
  • Include our license in the classifiers section as well
  • Introduce a long_description; convention here seems to be reading in your README

While I'm here, remove an extra space from imports and suppress the pylint error on using exec()

This file is now 100% first class awesome A+ python2 (with 3 compat)

Test Plan

Ran setup.py build and setup.py install

Diff Detail

Repository
rMSCA Scap
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
demon created this revision.Dec 6 2017, 10:52 PM
Restricted Application added a reviewer: mmodell. · View Herald TranscriptDec 6 2017, 10:52 PM
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 6 2017, 10:53 PM
mmodell accepted this revision.Dec 7 2017, 5:01 AM
This revision is now accepted and ready to land.Dec 7 2017, 5:01 AM
This revision was automatically updated to reflect the committed changes.