Skip to content

Commit

Permalink
fix: replace halo with yaspin (#19)
Browse files Browse the repository at this point in the history
* fix: replace halo with yaspin

* fix: move yaspin to the correct group

* fix: use text instead of write
  • Loading branch information
johnson2427 authored Jun 4, 2024
1 parent b25db81 commit f45d13f
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
10 changes: 6 additions & 4 deletions fief_client/integrations/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,8 @@
import urllib.parse
import webbrowser

from halo import Halo
from yaspin import yaspin
from yaspin.spinners import Spinners

from fief_client import (
Fief,
Expand Down Expand Up @@ -233,8 +234,9 @@ def authorize(
)
webbrowser.open(authorization_url)

spinner = Halo(
text="Please complete authentication in your browser.", spinner="dots"
spinner = yaspin(
text="Please complete authentication in your browser.",
spinner=Spinners.dots,
)
spinner.start()

Expand Down Expand Up @@ -263,7 +265,7 @@ def authorize(
)
self._save_credentials(tokens, userinfo)

spinner.succeed("Successfully authenticated")
spinner.ok("Successfully authenticated")

return tokens, userinfo

Expand Down
4 changes: 2 additions & 2 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ module = "jwcrypto.*"
ignore_missing_imports = true

[[tool.mypy.overrides]]
module = "halo.*"
module = "yaspin.*"
ignore_missing_imports = true

[tool.ruff]
Expand Down Expand Up @@ -113,7 +113,7 @@ flask = [
]

cli = [
"halo",
"yaspin",
]

[project.urls]
Expand Down

0 comments on commit f45d13f

Please sign in to comment.