Closing a PR should not close linked issues #66741
Replies: 21 comments 8 replies
-
Adding myself to this. I would also like to be able to disable auto-closing of issues. I work on PyMuPDF, and we do not want an issue to be marked as fixed until a release has been made that contains the fix. This avoids confusion for users, and simplifies our release process. Projects differ in how they want to handle things like this, so i find it surprising that Github is forcing the same behaviour on all projects, regardless of size or organisation. I don't personally mind what the default behaviour is, but it really needs to be configurable. |
Beta Was this translation helpful? Give feedback.
-
I agree this is an important thing for teams. The ability to link a PR without auto-closing it would be a great addition and something our team would definitely find very useful. Not every PR solves an issue completely - perhaps there is QA to do, perhaps there are manual steps, etc. There are many cases I can think of where the issue shouldn't be closed immediately and this should be the only option.
|
Beta Was this translation helpful? Give feedback.
-
Just throwing another comment in for support of this, I can't believe that GitHub is so prescriptive with this, expecting that everyone wants to close a linked issue as soon as they close a PR... Please just give us the option to turn off the automatic closing! |
Beta Was this translation helpful? Give feedback.
-
One more use case, when merging a PR shouldn't close the issue. To resolve some issues, from time to time I need more than one PR (usually in different repositories). So it's "slightly" inconvenient to reopen the issue each time a linked PR is merged. |
Beta Was this translation helpful? Give feedback.
-
+1 to this feature request |
Beta Was this translation helpful? Give feedback.
-
My use-case is that we have a While I could avoid linking them, often the developers include screenshots and context in the PR that is not in the original issue, and it's nice to be able to allow testers to see that context for their testing. |
Beta Was this translation helpful? Give feedback.
-
The behavior seems to have recently worsened. I am getting issues closed even if I don't link them to the PR, just because my commit format includes the issue identifier to trace the commits in the main branch after merge and squash. |
Beta Was this translation helpful? Give feedback.
-
Agreed, this would be great. We have fairly large user stories which require multiple PRs, and having them be linked but not closed would be awesome. Also having a magic word would require less manual work which might be missed. |
Beta Was this translation helpful? Give feedback.
-
We have a Quality Assurance (QA) team that reports bugs via issues on GitHub projects. Whenever a bug is identified, the QA team creates an issue and the developers later link the issue with their relevant Pull Requests (PRs). After the PRs are merged, the issue is closed. However, the QA team still needs to retest the solution. In this scenario, we would like to configure the issues to remain open even after the linked PRs are merged. This would enable the QA team to retest the solution with ease. Thank you in advance! |
Beta Was this translation helpful? Give feedback.
-
I don't see anyone highlighting the need for creating this change also for Github Enterprise Server, so I'll add that from me :) |
Beta Was this translation helpful? Give feedback.
-
Looking forward to getting this addressed in a better, more elegant way. |
Beta Was this translation helpful? Give feedback.
-
+1 It's like a car with an automatic parking feature. Convenient for fitting into easy spots without effort, but in some situations, you might prefer to control the parking manually to ensure it's done according to specific needs or constraints. Please make this an option, I dont really care what the default is as long as I can change it. |
Beta Was this translation helpful? Give feedback.
-
Also note that the keyword recognition is somewhat dumb and can be triggered accidentally. For example this trivial NFC commit closed an issue because I accidentally used the word "fix". Moreover, the after I reopened it the PR was closed again when the same commit was pushed to a private copy of the repository. This is outright buggy behavior in my opinion... |
Beta Was this translation helpful? Give feedback.
-
Please make this an option to auto-close an issue when a PR is merged. +1 to a previous post on making auto-close off by default and providing the option to auto-close when a developer links the PR to the issue. |
Beta Was this translation helpful? Give feedback.
-
+1 make auto-close configurable. That's annoying, I want to have my Project item to move the testing when my PR is merged... not close that issue :( |
Beta Was this translation helpful? Give feedback.
-
What's annoying is the lack of response from GitHub.... Sorry, I had to vent a little. |
Beta Was this translation helpful? Give feedback.
-
I've been looking for a solution on this to no avail.. Kinda far from the original problem but I though I'd comment this here in case someone lands here the same way I did and learns from it. |
Beta Was this translation helpful? Give feedback.
-
I've stopped linking my issues unless I really want the current behavior (which is almost never). What I think Github devs could do, which may be a more in the "low hanging fruit" category, is to summarize (on the right) which pull-requests have cited this issue (and vice-versa: which issues a pull request cites). Wouldn't address all use-cases, but it would help testers easily access the context they need from those other issues/PRs rather than searching through the conversation history. |
Beta Was this translation helpful? Give feedback.
-
We have now sort of solved this problem by just not using GitHub issues / projects anymore but moved to Jira. Much better experience overall and not looking back. And a great GitHub integration in Jira including PRs, commits, build status, and deployments. Notice the closed/merged PR but the issue is still "In Progress". Can be that easy. Not really an option for open source projects, but if you have private repos, give it a try. Maybe this is just GitHub realizing that company customers will at some point use Jira anyway and oss projects don't really have a choice - so they're just ignoring the problem because there's nothing to win, even if they fix it. I mean, 4 years later and not even a switch to disable the default action. It can't be that difficult. |
Beta Was this translation helpful? Give feedback.
-
Context
This issue serves as an urgent follow-up to the discussion initiated here: GitHub Community Discussion #23476. Despite considerable community support, there has been no official response for three years. The lack of attention may be attributed to the discussion being marked as answered by @AndreaGriffiths11.
Current Behaviour
Rationale for Reconsideration
The current behaviour is incongruent with the workflows of numerous development teams for several reasons:
Inadequacy of Suggested Workarounds
Previous discussions proposed linking the PR to the issue without using specific linking keywords. This approach, however, fails to display the PR in a GitHub Project card, negating the benefits of linking.
Proposed Alternatives
The issue can be addressed through multiple avenues:
Community Sentiment
The following comments and discussions underscore the urgency and support for revisiting this issue:
We urge the GitHub team to address this issue promptly to align the platform's functionality with the diverse needs of the development community.
Disclaimer: Yes, ChatGPT wrote this.
Beta Was this translation helpful? Give feedback.
All reactions