netlify / netlify/cli

netlify-watch: remove spinner by default or via option

Open
#5,301 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

good first issue type: feature
Dominant language
TypeScript
Stars
1.9k
Forks
474
Avg merge
23h 30m
Merged PRs (30d)
53

Description

Which problem is this feature request solving?

As a developer living in a terminal shell most of the time,
I want $ netlify watch to be silent unless there is progress on the deployment,
so that I know when to return to this terminal window/tab/tmux-window for progress updates.

https://user-images.githubusercontent.com/175286/206479201-d38a1f81-d121-4d6c-bd06-0d52acead91f.mov

Currently the watch command have a pretty cool spinner (video) while waiting. However I find that I usually switch away from the terminal where I ran the netlify-watch in and would expect that it only prints something when there is some useful progress to watch (most likely deployment finished).

Screen Shot 2022-12-07 at 16 57 02

Many terminal emulators have some way to show that something has happened since last time it was in the foreground. For example tmux marks windows that has new output (screenshot: window 4 has new output from netlify watch : the spinner moved)

Describe the solution you'd like

Possible solutions

  • Remove the spinner and replace with a static icon/char/text
    • Possibly allow with with an option --with-spinner
  • Add an option --no-spinner

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at the netlify watch command entry point and trace where its spinner output is produced. Compare the requested default behavior with the proposed --with-spinner and --no-spinner options; done means the command avoids spinner output by default or follows the selected option consistently.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
cli
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.