Disable Thot bot (sesheta) on this project

Question

Disable Thot bot (sesheta) on this project

frenzymadness opened this issue 2 years ago · 10 comments

I'd like to disable Thot bot (sesheta) on this project. In my opinion, it generates unnecessary comments and does not add any value to my workflows.

@goern what do you think?

Answer 1 · 2022-07-26T05:39:36.000Z

I'm good with hat, if you could give a little insight what you like/dont like, what you would like to see, etc.

@Gkrumbach07 @mayaCostantini
/kind feature
/priority critical-urgent
/triage accepted

Answer 2 · 2022-07-26T07:55:18.000Z

I'm just used to managing my repositories manually. For example, I don't like the auto-closing of issues. Also, every new PR receives an immediate comment from the bot that the issue is not approved, which is obvious for new PRs. Github has its own standard way for approving of PRs and I like it more than the special commands. One last thing is that the comment from the bot in the latest PR says that I should request a review from myself.

Answer 3 · 2022-07-26T07:57:19.000Z

/sig user-experience

Answer 4 · 2022-08-02T10:08:16.000Z

Hey @frenzymadness I think you are talking about all the Prow activity you see on issues.

wrt to approving PR, the Prow plugin and GitHub are equivalent, you can use one or the other with the same results. so if you like to click approve on the github web UI, feel free, it will also add the 'approved' label. I wonder why you do the additional manual step of merging by yourself, if a PR is approved, the bot will merge it for you ;)

The initial comment of the bot, that a PR needs approval and who is responsible for review/approve might be obvious, but especially to new contributors, it might be helpful.

Nevertheless, if you can not live with these helpers, feel free to work with @Gregory-Pereira to reconfigure Prow for this repo.

Answer 5 · 2022-10-17T12:49:52.000Z

any update on this @frenzymadness?

Answer 6 · 2022-10-24T09:08:19.000Z

@Gkrumbach07 I still think I don't need the bot here. But I'm not alone anymore so what do you @VannTen think?

Answer 7 · 2022-10-24T09:30:14.000Z

The lifecycle labels and auto-closing should be disabled now for all repos under thoth-station/ Regarding the tide stuff (approved and lgtm labels and automerging) I'd rather keep them, in particular the lgtm (can only be put by someone else than a PR author if I'm correct) works pretty well to be sure to have a least two set of eyes on stuff going into the main branch.

Answer 8 · 2022-11-07T13:53:22.000Z

@frenzymadness @VannTen Is there any other actions to be done regrading the Thoth bot on this repo? If not I will close this issue

Answer 9 · 2023-01-09T10:16:16.000Z

The current state should be :

no lifecycle handling by prow.
prow auto-merge PR when they have approved + lgtm labels.

If you're ok with that, we can close.

Answer 10 · 2023-01-11T14:52:12.000Z

Okay. I'm closing this now. Thanks to all of you.