On Tue, Jun 29, 2021 at 10:38 AM <esmeraldagar...@byom.de> wrote: > I just stumbled upon the following issue and subsequent pull request. It > is a very small bugfix. There is currently a bug in Python and this pull > request fixes it. It's not a new feature or an enhancement, it is a bugfix! > Yet, it doesn't get reviewed, nor merged. And this has been going on since > March 2017. Why not just merge this? It's not like it's huge or obstructing > or obfuscating the current code base? There's always time to write an > improvement or a complete rewrite of this module feature later for an > upcoming minor release. But if there is currently a bug in Python and the > bugfix is available - why doesn't it get merged? > > https://github.com/python/cpython/pull/4819
For starters, the PR is closed in favor of another issue that has reviews and a discussion, but even the smallest change like that requires a lot out of a reviewer. Looking at that change, I don't personally know that it's correct, so I'd have to take the time to figure out that it's correct. It includes no tests, so I certainly don't trust that it's correct, so it looks incomplete to me. Time is irrelevant here—there's no need to rush things because a change appears small. What if that one line change is even more wrong than before? I have merge access and if I just said "ah it's a small PR and it's been open for a while, I'll just merge it for them," any change to Python has the possibility to affect a huge amount of people. When I got the shutil.which feature merged, the PR had been open for I believe 11 years and it was mostly complete in the initial patch outside of a few small issues, and the change itself wasn't a lot of code. To just have merged it because it was open for 11 years would have been the wrong thing to do. It needed to cover some things it didn't initially cover, it needed tests and documentation, and it wasn't merged until it was completed and properly reviewed. If this doesn't get fixed, doesn't that mean the Python review process is > flawed? Sure, Python is an open source project and many people just work in > their free time. But this doesn't really apply here, does it? The bugfix is > available. Only a review is required. All this is happening while new > features get added to Python with every new minor version. While the bug is > allowed to live there. Please help me understand how this can happen. > "Only a review is required" is vastly understating the value of code reviews. Almost anybody can write a one line fix, but is it the right fix? Does it cover all of the cases it needs to? Is adding "manager_owned=False" correct or should something else actually be done? Who knows and is available to understand the impacts of this change? So does this mean the review process is flawed? I would say no, the _review process_ is maybe working as expected—the linked PR was incomplete and wasn't merged, another PR has come up, and there's discussion on it including a comment about tests and one about familiarizing with the code. The process of finding humans who are willing and able to do this work—currently for free—is possibly broken, possibly working as expected, and overall is a significantly harder problem to fix than most anything involved with open source software. I love Python. No hard feelings. But this is really bugging me and I can't > help but feel disappointed. > The good thing is that you paid nothing for this, so being disappointed is something you can fix. If you would like more value out of it or to speed up the process, you can provide your own reviews. Reviewing code is immensely valuable and helps so many people—the core developers, the users, and yourself. Alternatively, paying people to do the reviews is also possible. _______________________________________________ > Python-Dev mailing list -- python-dev@python.org > To unsubscribe send an email to python-dev-le...@python.org > https://mail.python.org/mailman3/lists/python-dev.python.org/ > Message archived at > https://mail.python.org/archives/list/python-dev@python.org/message/TTJAVTPF6RYO63GTBSTXFJG3IVCYPHXT/ > Code of Conduct: http://python.org/psf/codeofconduct/ >
_______________________________________________ Python-Dev mailing list -- python-dev@python.org To unsubscribe send an email to python-dev-le...@python.org https://mail.python.org/mailman3/lists/python-dev.python.org/ Message archived at https://mail.python.org/archives/list/python-dev@python.org/message/5PYTF4BEKS3H2AZX6ZQXKIQM5SYTOH7H/ Code of Conduct: http://python.org/psf/codeofconduct/
