!VRULIdgoKmKPzJZzjj:nixos.org

Nix Hackers

871 Members
For people hacking on the Nix package manager itself188 Servers

Load older messages


SenderMessageTime
28 Oct 2025
@roberthensing:matrix.orgRobert Hensing (roberth) Mic92: ^ 18:21:14
@roberthensing:matrix.orgRobert Hensing (roberth)it's opt-in but quite noisy nonetheless18:21:37
@roberthensing:matrix.orgRobert Hensing (roberth)could it be configured not to post that comment?18:22:04
@fzakaria:one.ems.hostfzakariaor if the template for a PR could include whatever you need to toggle it18:24:24
@fzakaria:one.ems.hostfzakariaand we just exclude it from posting then -- so people can still find it18:24:33
@xokdvium:matrix.orgSergei Zimmerman (xokdvium)
In reply to @roberthensing:matrix.org
could it be configured not to post that comment?
Yeah, that was the intention I believe. It can be configured in-repo like mergify. Feel free to send a pr
18:31:51
@shine:proqqul.netTaeer Bar-YamIn what sense is it opt-in? It ran on my MR and I didn't opt in20:02:06
@joerg:thalheim.ioMic92
In reply to @roberthensing:matrix.org
Mic92: ^
It was only doing auto review until we had a chance to turn off. I had to go through an org owner though because, i can't configure anything.
20:06:03
@shine:proqqul.netTaeer Bar-Yam
  1. Will other people be able to request it's reviews on my PRs?
  2. Will this be socially acceptable?
20:07:22
@shine:proqqul.netTaeer Bar-Yam *
  1. Will other people be able to request its reviews on my PRs?
  2. Will this be socially acceptable?
20:07:37
@joerg:thalheim.ioMic92Maintainers can request it. The way i do it usually is to triage the responses myself upfront. 20:09:45
@xokdvium:matrix.orgSergei Zimmerman (xokdvium)@Mic92 can we also make it stop spamming “auto-review disabled” please20:13:50
@joerg:thalheim.ioMic92I'll ask. I had a person reaching out to me. 20:18:34
@joerg:thalheim.ioMic92You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.20:20:22
@joerg:thalheim.ioMic92It says this in the message actually 20:20:50
@xokdvium:matrix.orgSergei Zimmerman (xokdvium)Yeah I saw that. Just away from the keyboard20:21:26
@joerg:thalheim.ioMic92I might have to bother infinisil again though to allow per-repo settings.20:25:48
@joerg:thalheim.ioMic92Not sure if our configuration yaml is picked up20:26:05
@joerg:thalheim.ioMic92 lassulus: can also do this I think by going to https://app.coderabbit.ai and go to "repositories" -> nix -> Use Organization Settings 20:26:50
@shine:proqqul.netTaeer Bar-YamI'm actually quite upset about the addition of LLM code review bullshit to the CppNix repo and might stopping contributing if I were expected to read it, much less respond. I'm not sure how to handle this situation. I'm obviously not in charge, and I'm not trying to threaten or anything. But I figured I should say something at the outset rather than just disappearing or waiting until the norms have ossified.20:27:00
@joerg:thalheim.ioMic92 Taeer Bar-Yam: even I hide the message after I read it? 20:27:32
@joerg:thalheim.ioMic92 * Taeer Bar-Yam: even If hide the message after I read it? 20:30:17
@joerg:thalheim.ioMic92 * Taeer Bar-Yam: even If I hide the message after I read it? 20:30:22
@lassulus:lassul.uslassulusI can take a look20:31:15
@lassulus:lassul.uslassulusimage.png
Download image.png
20:31:48
@lassulus:lassul.uslassulusthis is what is set rn20:31:50
@lassulus:lassul.uslassulusshould I change smth?20:31:59
@joerg:thalheim.ioMic92 lassulus: untoggle 20:32:13
@joerg:thalheim.ioMic92so it uses our settings20:32:20
@lassulus:lassul.uslassulusany settings I need to set?20:32:51

Show newer messages


Back to Room ListRoom Version: 6