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

RFE: Silent mode #885

Open
sils opened this issue Jun 29, 2015 · 12 comments
Open

RFE: Silent mode #885

sils opened this issue Jun 29, 2015 · 12 comments
Labels

Comments

@sils
Copy link
Contributor

sils commented Jun 29, 2015

People are getting a lot of emails in our project. GitHub's bad at regulating that. It would be nice to not have PR's and issues cluttered with unneeded comments. E.g. for a merge I get an email from github on merge anyways that rultor merged that stuff. Further flooding with "hey I'm doing a simple merge now like you requested" and "yep that worked" messages is not needed IMO. Zero messages on success, one message on failure would be my aim.

If I want to see the log one would probably need an additional log command or a parameter to the merge command (verbose=true).

Same probably also applies to releases.

@yegor256
Copy link
Owner

@sils1297 make sense, thanks. how about this (in .rultor.yml):

merge:
  quiet: true

false by default. If it's set to true, you see no "success" messages, only when failed.

@sils
Copy link
Contributor Author

sils commented Jun 30, 2015

@yegor256 I'd rather prefer a global setting for all commands plus also avoiding the command acknowledgements. I.e. the merge command should just merge with no comments at all.

@yegor256
Copy link
Owner

@sils1297 yeah, make sense. so it will be a global option (false by default):

quiet: true

@yegor256 yegor256 added the bug label Jun 30, 2015
@yegor256 yegor256 self-assigned this Jun 30, 2015
@alex-palevsky
Copy link
Contributor

@sils1297 since there is no milestone yet I set it to "2.0"

@alex-palevsky alex-palevsky added this to the 2.0 milestone Jun 30, 2015
@alex-palevsky
Copy link
Contributor

@yegor256 this is your task now,help us

@sils
Copy link
Contributor Author

sils commented Jul 1, 2015

@yegor256 just an idea about how you can make it even cooler - provide the information while not doing comments: use the status API to set a rultor PR status.

@yegor256
Copy link
Owner

@alex-palevsky assign someone else pls

@alex-palevsky
Copy link
Contributor

@alex-palevsky assign someone else pls

@yegor256 -30 points to your rating :(

@alex-palevsky
Copy link
Contributor

@alex-palevsky assign someone else pls

@yegor256 got it, someone else will be assigned soon

@original-brownbear
Copy link
Contributor

@alex-palevsky this is postponed.

@alex-palevsky
Copy link
Contributor

@alex-palevsky this is postponed.

@original-brownbear OK, I put "postponed" label here

@alex-palevsky
Copy link
Contributor

@alex-palevsky this is postponed.

@original-brownbear got it, someone else will be assigned soon

@yegor256 yegor256 removed their assignment Sep 10, 2022
@yegor256 yegor256 removed this from the 2.0 milestone Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants