[PR #2007] [MERGED] Important: wrong PR merge commit ID saved #16177

Closed
opened 2025-11-02 12:04:02 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-gitea/gitea/pull/2007
Author: @lafriks
Created: 6/18/2017
Status: Merged
Merged: 6/19/2017
Merged by: @lunny

Base: masterHead: patch-2


📝 Commits (1)

  • 73935fb Wrong PR merge commit ID saved

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 models/pull.go (+1 -1)

📄 Description

When merging PR changes that were made from forked repository last commit ID is taken from fork not base repository but for target branch, so it will point to wrong commit ID


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/go-gitea/gitea/pull/2007 **Author:** [@lafriks](https://github.com/lafriks) **Created:** 6/18/2017 **Status:** ✅ Merged **Merged:** 6/19/2017 **Merged by:** [@lunny](https://github.com/lunny) **Base:** `master` ← **Head:** `patch-2` --- ### 📝 Commits (1) - [`73935fb`](https://github.com/go-gitea/gitea/commit/73935fbd84da0870960f1bc953bee8549be8e403) Wrong PR merge commit ID saved ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `models/pull.go` (+1 -1) </details> ### 📄 Description When merging PR changes that were made from forked repository last commit ID is taken from fork not base repository but for target branch, so it will point to wrong commit ID --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2025-11-02 12:04:02 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#16177