Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce noise in gps rr #258

Open
Alizter opened this issue Oct 20, 2023 · 0 comments
Open

Reduce noise in gps rr #258

Alizter opened this issue Oct 20, 2023 · 0 comments

Comments

@Alizter
Copy link
Contributor

Alizter commented Oct 20, 2023

When doing gps rr we typically get a very noisy output and this can be overwhelming to users. When using gps brr or eventually the proposal in #255, this issue gets multiple times worse. There are two main issues here:

  1. We output the output of Git. This includes all the noisy output about pushing to remotes.
  2. We have a very noisy hook message.

The second is not a problem once you've setup hooks, so perhaps that can be dealt with at a later time. The first is more immediately concerning for us.

I propose to change the output so that we only print relevant information for the use. Pretending that gps rr can batch, it would look like this:

gps rr 1-2 3 4-6
Successfully pushed 1-2
Isolation error on 3
Successfully pushed 3-6

We can do this by capturing the output of git and perhaps displaying it later if something goes wrong. Most of the time it is irrelevant however.

@github-project-automation github-project-automation bot moved this to Triage in git-ps-rs Oct 20, 2023
@drewdeponte drewdeponte moved this from Triage to Accepted in git-ps-rs Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant