CLI: add Prompt.spinner #5404
CLI: add Prompt.spinner #5404
Conversation
|
|
Love it! 🎉 |
|
Awesome!! Would be really cool to also support concurrency / multiple spinners.. 👀 |
| import { Console, Effect } from "effect" | ||
|
|
||
| // Demonstration of success, failure, and custom final messages | ||
| const program = Effect.gen(function*() { |
There was a problem hiding this comment.
Looks like there are a bunch of compilation errors in the example here.
| export const spinner = <A, E, R>( | ||
| message: string, | ||
| effect: Effect.Effect<A, E, R>, | ||
| options?: SpinnerOptions<E, A> | ||
| ): Effect.Effect<A, E, R | Terminal.Terminal> => |
There was a problem hiding this comment.
Let's make this signature dual - I can help with that if you're unsure, sometimes getting the signature right with an optional third param can be tricky.
| const indexRef = yield* Ref.make(0) | ||
| const exitRef = yield* Ref.make<Option.Option<Exit.Exit<A, E>>>(Option.none()) |
There was a problem hiding this comment.
Since these are only ever accessed / modified within the generator, they can be plain mutable variables.
| const indexRef = yield* Ref.make(0) | |
| const exitRef = yield* Ref.make<Option.Option<Exit.Exit<A, E>>>(Option.none()) | |
| const index = 0 | |
| const exit: Exit.Exit<A, E> | undefined = undefined |
github-project-automation
bot
moved this from Discussion Ongoing
to Waiting on Author
in PR Backlog
kitlangton
force-pushed
the
feat/spinner
branch
from
7d63555 to
731b16f
Compare
4 participants
Type
Description
Added a spinner to
Promptthat animates a spinner while an effect is running.The message is updated when the effect completes successfully or fails.