manrajgrover / halo

💫 Beautiful spinners for terminal, IPython and Jupyter
MIT License
2.89k stars 147 forks source link

BUG FIX: Support pylint on Python 3 #104

Closed theY4Kman closed 5 years ago

theY4Kman commented 5 years ago

Description of new feature, or changes

This PR expands the base64-encoded pylint init hook into its full script (because INI files can do that; idk what this guy was smoking), and converts its usage of execfile(path) to exec(open(path).read()) to support execution under Python 3.

NOTE: backslashes / continuation operators were used for indented blocks, 'cause though INI files can support multiple lines (though blank lines are removed by default), I had no idea how to also have it respect leading whitespace.

Checklist

Related Issues and Discussions

Mentioned here: https://github.com/manrajgrover/halo/issues/102#issuecomment-431643972

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 299


Totals Coverage Status
Change from base Build 291: 0.0%
Covered Lines: 272
Relevant Lines: 289

💛 - Coveralls