Copilot in nixpkgs pull requests?

It seems that someone enabled Copilot on nixpkgs: qownnotes: add aspell support by pbek · Pull Request #448223 · NixOS/nixpkgs · GitHub

Can we please disable it again?

7 Likes

Any individual user can request Copilot review of a PR, and as far as I can tell there’s no way to block this at the org/repo level. So it’s not something we enabled in nixpkgs, and thus cannot disable.

1 Like

Do you know a way to block it?
Apparently I can only “hide” the message, but not block the “account” altogether.

Ninja edit: Ah, okay, thanks for the link

Hi @musicmatze,

Please voice your concerns without being a prick while remaining kind.

I’m not a fan of using LLMs like Copilot for this either, but @pbek has not violated any rule by using a tool that the platform nixpkgs is maintained on, actively encourages.
He is the maintainer of QOwnNotes and is co-maintainer of the qownnotes package, so if there are no rules that say otherwise, it is at his discretion to evaluate using Copilot for this in nixpkgs as well.

@pbek Please reconsider using it, as this shows that the comment the LLM provided was not adding anything to the discussion, since the point you made in the PR description were sufficient as is for me as a reviewer to understand. The societal concerns of using LLMs for everything in this case definitely outweigh the benefit IMO.

Thanks

4 Likes

Could we refrain from this kind of language? It’s not helpful, and I don’t believe poster was a prick either.

2 Likes

@adamt What kind of language do you mean? How is that not a reasonable request?

Edit: I see that it might have been unclear that I did not refer to the post on Discourse, but to the comment on the PR on GitHub. I have no issue whatsoever with the post @musicmatze made here and think it is a good idea to discuss this.

IMO stuff like this can be easily discussed without going directly at someone. If you do not believe that the way @musicmatze voiced their concern, that’s okay, but I voiced my opinion that one can remain kind while still being adamant that such tools shouldn’t be used.

A “please don’t do this” goes down easier than a “who the fuck does such bullshit”, doesn’t it?

1 Like

I think calling someone “a prick” is in itself “going directly at someone”. I appreciate the desire to tone down hostile and reactionary language, but I’d encourage not escalating the situation in an attempt to calm things down.

14 Likes

Sure, sorry @musicmatze. Did not consider that the word prick itself is offensive as I watched a lot of British television recently.
Edited the reply transparently.

1 Like

Yeah, I fear that was me. I saw a link to request a review by copilot in the reviewers list and clicked to see if it does something valuable. It looks like it does a summary of the changes in a comment. Seems like that’s a new GitHub feature. :person_shrugging:t2: Now that I know what it does, I will not click it again here.

I think it was only a matter of time until Microsoft integrated Copilot into GitHub.

5 Likes

Yeah, my changes were small enough and easy to understand. But I wonder what it does for big PRs. :thinking: Unfortunately, you don’t get a preview of what will be posted, nor can you delete or hide the report (at least I can’t).

A committer can hide it, but unfortunately the person who requested isn’t granted that permission.

I see, thank you for the explanation!

Thanks to everyone involved in this thread!

@totoroot I didn’t mean to attack @pbek personally, of course (and I posted on github as well to make that clear), sorry if it was too easy to misinterpret.

3 Likes