Skip to content

Don't attempt to close issue if already closed#9696

Merged
lunny merged 2 commits into
go-gitea:masterfrom
guillep2k:fix-9695
Jan 11, 2020
Merged

Don't attempt to close issue if already closed#9696
lunny merged 2 commits into
go-gitea:masterfrom
guillep2k:fix-9695

Conversation

@guillep2k
Copy link
Copy Markdown
Member

@guillep2k guillep2k commented Jan 10, 2020

Fixes #9695

Couldn't reproduce, however, neither with a commit message or a text reference. Tested with master (0c0ebe49ea4fd0d3ecc2aac9dfe2d7ce64381145) and v1.11.0-rc1.

Update

I can reproduce if the issue is closed from both the commit message and a text reference. This PR fixes that.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jan 10, 2020

Codecov Report

Merging #9696 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #9696      +/-   ##
==========================================
- Coverage   42.25%   42.24%   -0.02%     
==========================================
  Files         590      590              
  Lines       78005    78004       -1     
==========================================
- Hits        32961    32952       -9     
- Misses      41001    41012      +11     
+ Partials     4043     4040       -3
Impacted Files Coverage Δ
routers/api/v1/repo/repo.go 69.55% <ø> (-0.04%) ⬇️
models/issue.go 55.02% <100%> (ø) ⬆️
services/pull/check.go 50.67% <0%> (-2.71%) ⬇️
models/webhook.go 69.39% <0%> (-1.07%) ⬇️
routers/repo/view.go 38.59% <0%> (-0.88%) ⬇️
models/error.go 31.6% <0%> (+0.55%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4935b47...d0fb57c. Read the comment docs.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 10, 2020
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jan 10, 2020
@6543
Copy link
Copy Markdown
Member

6543 commented Jan 10, 2020

I would say backport ;)

@guillep2k
Copy link
Copy Markdown
Member Author

I would say backport ;)

I always wait for the approvals and merge before backporting. You never know... 😁

@6543
Copy link
Copy Markdown
Member

6543 commented Jan 10, 2020 via email

@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 Jan 11, 2020
@lunny lunny merged commit 705b1e4 into go-gitea:master Jan 11, 2020
@lunny
Copy link
Copy Markdown
Member

lunny commented Jan 11, 2020

@guillep2k Please send back port to v1.11

@lafriks lafriks added the backport/done All backports for this PR have been created label Jan 11, 2020
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport/done All backports for this PR have been created 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.

Error 500 when closing already closed issue from pull request

7 participants