Live data from Hacker News

GitHub discusses giving maintainers control to disable PRs

github.com

71–75 of 75 posts

Re: GitHub discusses giving maintainers control to disable PRs

#71
post #67

Earlier quoted context omitted.

There is no confusion about that. I think the confusion is repo being used for both 1) the actual remote git repo hosted by GitHub and 2) the “repo” at GitHub.com/username/project. I know that opening a PR does not affect the first, but it very much affects the second. For my project both are mine, and just as I have the right to ignore a PR and I have the right to reject any after they’re opened, so too do I have th…

> So what? Who or what states they are entitled to have their changes visible as a request on my repo? You did, by making the repo public. You didn't make a release tarball public, you made git repository, designed for collaborating on code changes public. In this case, other users of github (including the general public) are exposed to a repository and all that entails. If you have no intention to let others do PRs,…

> you just need to host public tarballs of your source code.

Except now I can do it all on one place.

> public repo = public/open PRs, because that's how a vcs works.

No it’s not. A PR is not a feature of git the vcs, it’s a feature of GitHub the website.

> other have the right to make community corrections

No, others have the ability. They are not an unalienable right to do so. Likewise they don’t have a right on my GitHub repo.

You keep falling back to “because they can, they have a right to”, which I think is obviously incorrect.

And I disagree that this is similar to notes on Twitter. It is comments on YouTube or replies on Twitter; they are by and large used by people to add their opinion on the topic.

Re: GitHub discusses giving maintainers control to disable PRs

#72
post #6

Earlier quoted context omitted.

yeah, I thought they were going to provide some sort of rationale as to why they've never implemented this. instead this post just basically goes "yeah, you guys have been asking for this feature for 10 years, and... it's a good idea! let's do it."

Honestly, it's not an area where there has been consensus on when we talk with maintainers. Some folks worry about that reducing the very nature of open source collaboration. We've had the ability to temporarily disable PR's for a while for maintainers but we felt like it was time to look at this again and see what folks think.

> Some folks worry about that reducing the very nature of open source collaboration.

Collaboration on repos where the authors explicitly don't accept PRs and are going to auto close or ignore them? I don't get it - it's not like you're going to force opensource on anyone.

Re: GitHub discusses giving maintainers control to disable PRs

#74
post #45

Hey everyone, I'm a PM on the GitHub team building this feature. Really appreciate all the feedback coming in and want you to know that we're reviewing it carefully so we can figure out the best path forward. Disabling PRs is just the first step in giving maintainers more control over their PR experience. We're exploring several longer-term ideas which you can learn more about in this discussion: https://github.com/o…

Hi, There are still some projects and community which requires external contributor to contribute to their open source projects to grow open source community and people who are beginner in open source.

these communities and projects are still vulnerable to the AI and spam PRs. for those kind of people how can this help.

My suggestion on this take you should add an option to maintainer to flag the user if they are doing spam and ai slop so if it exceed to a x number it will ban the user.

I know this is against the open source philosophy but still it will prevent the open source community from being polluted.

Re: GitHub discusses giving maintainers control to disable PRs

#75
post #10

Earlier quoted context omitted.

Wow, what is the context for all of these spam PRs?

Tldw: a popular YouTube video on “how to open a PR on GitHub” by an Indian channel (targeting Indian audiences) showed how to add their name to a PR step by step. The rest is just the scale of the Indian population in action. I hope the maintainers of expressjs can rest easy

I am from India and i would like to apologize for this matter.

:(

Post reply on HN