Closed laglangyue closed 8 months ago
then add vcs.xml
<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
<component name="VcsDirectoryMappings">
<mapping directory="$PROJECT_DIR$" vcs="Git"/>
</component>
<component name="IssueNavigationConfiguration">
<option name="links">
<list>
<IssueNavigationLink>
<option name="issueRegexp" value="#(\d+)"/>
<option name="linkRegexp" value="https://github.com/apache/seatunnel/pull/$1"/>
</IssueNavigationLink>
</list>
</option>
</component>
</project>
@mdedetrich @pjfanning ping, I can't access the setting tab.
You get the PR link if you squash merge.
I see, so I just need do a squash merge !
Do note that the PR link in the git message is largely irrelevant because the github ui allows you to search by the commit hash and when you do this it will also provide the PR behind that commit hash, even though I do admit that it is a nice touch.
Another thing to keep note of is that technically speaking according to ASF policy, github is a convenience and hence ASF projects shouldn't be tied to github. There is gitbox for that.
Do note that the PR link in the git message is largely irrelevant because the github ui allows you to search by the commit hash and when you do this it will also provide the PR behind that commit hash, even though I do admit that it is a nice touch.
Another thing to keep note of is that technically speaking according to ASF policy, github is a convenience and hence ASF projects shouldn't be tied to github. There is gitbox for that.
Infact, we won't actually add a link to the git message, we just added the PR ID
.
such as this, the idea vcs plugin will translator the PR ID
to a link,This is a convenient operation to review history.
as pjfanning said, use squash merge
would get the pr-id
I understand, my point is that even the PR link (i.e. the #285
which you highlighted in the screenshot) is just a minor convenience if you copy the git hash of the commit and paste it in the search bar of the github repo, github will bring you to the original PR.
like this sreenshot, which is from apache seatunnel
this link is not added by developer, and it's automatic, maybe by github. this link can help us to trace history and reasons for changes and obtain more information