Skip to content

Conversation

@6543
Copy link
Member

@6543 6543 commented Nov 9, 2020

The issue lay in determining the position of matches on a second run round
a commit message in FindAllIssueReferences.

Fix #13483
Backport #13486

The issue lay in determining the position of matches on a second run round
a commit message in FindAllIssueReferences.

Fix go-gitea#13483

Signed-off-by: Andrew Thornton <art27@cantab.net>
@6543 6543 added this to the 1.13.0 milestone Nov 9, 2020
@6543 6543 added type/bug issue/critical This issue should be fixed ASAP. If it is a PR, the PR should be merged ASAP labels Nov 9, 2020
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Nov 9, 2020
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Nov 9, 2020
@zeripath
Copy link
Contributor

zeripath commented Nov 9, 2020

@6543 Does 1.13 need a backport of #13468 to fix the intermittent ci issue?

@6543
Copy link
Member Author

6543 commented Nov 9, 2020

@zeripath why not ... I dont know how many backports we will make, but each time we do ... your patch could be usefull

@zeripath
Copy link
Contributor

zeripath commented Nov 9, 2020

Feel free to make a backport of it

@6543
Copy link
Member Author

6543 commented Nov 10, 2020

🚀

@techknowlogick techknowlogick merged commit ae49559 into go-gitea:release/v1.13 Nov 10, 2020
@6543 6543 deleted the backport_zeripath_fix-13483-error-in-reference-munching branch November 10, 2020 02:17
@go-gitea go-gitea locked and limited conversation to collaborators Dec 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

issue/critical This issue should be fixed ASAP. If it is a PR, the PR should be merged ASAP lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants