manrajgrover / halo

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

[#118] Removes cursor dependency #147

Closed manrajgrover closed 3 years ago

manrajgrover commented 3 years ago

Description of new feature, or changes

This PR removes cursor dependency. It replaces it with the code available in stackoverflow answer: https://stackoverflow.com/a/10455937/2692667

Checklist

Related Issues and Discussions

Fixes #118

People to notify

None

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 438


Changes Missing Coverage Covered Lines Changed/Added Lines %
halo/halo.py 28 30 93.33%
halo/cursor.py 5 26 19.23%
<!-- Total: 44 67 65.67% -->
Totals Coverage Status
Change from base Build 429: -6.0%
Covered Lines: 311
Relevant Lines: 356

💛 - Coveralls
coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 434


Changes Missing Coverage Covered Lines Changed/Added Lines %
halo/halo.py 28 30 93.33%
halo/cursor.py 5 26 19.23%
<!-- Total: 44 67 65.67% -->
Totals Coverage Status
Change from base Build 429: -6.0%
Covered Lines: 311
Relevant Lines: 356

💛 - Coveralls