manrajgrover / halo

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

Bug: placement='right' in API doesn't work #65

Closed hyeonhong closed 6 years ago

hyeonhong commented 6 years ago

Description

After adding placement='right' in the Halo constructor, the code gives an error that says "TypeError: init() got an unexpected keyword argument 'placement'". Am I doing something wrong here?

from halo import Halo
import time

spinner = Halo(text='Loading', spinner='dots', placement='right')
spinner.start()

time.sleep(3)  # Running time-consuming work

spinner.stop()

System settings

Error

Traceback (most recent call last): File "main.py", line 8, in spinner = Halo(text='Loading', spinner='dots', placement='right') TypeError: init() got an unexpected keyword argument 'placement'

Expected behaviour

The spinner should be placed on the right of text.

Steps to recreate

People to notify

manrajgrover commented 6 years ago

Hi @hshong9590 ,

Thanks for opening this issue. I believe this is happening because this feature is not released for now. I plan to make a release this week sometime. Can you check if you're facing this issue from master branch of this project?

manrajgrover commented 6 years ago

I've made a release with this feature. Please upgrade your version to 0.0.13 and see if it is fixed. Do let me know if you're still facing this issue.

hyeonhong commented 6 years ago

It works! Thank you, @ManrajGrover :+1:

manrajgrover commented 6 years ago

Thanks for confirming @hshong9590. Closing this.