Skip to content

Conversation

@techknowlogick
Copy link
Member

go panics otherwise with panic: interface conversion: error is git.ErrNotExist, not *git.ErrNotExist, thanks to Codeberg/Andi for reporting this.

go panics otherwise with `panic: interface conversion: error is git.ErrNotExist, not *git.ErrNotExist`, thanks to Codeberg/Andi for reporting this.
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Apr 23, 2021
@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 Apr 24, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #15604 (bccfe7c) into master (1b3dbdb) will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #15604      +/-   ##
==========================================
+ Coverage   43.88%   43.92%   +0.04%     
==========================================
  Files         678      678              
  Lines       81742    81742              
==========================================
+ Hits        35874    35907      +33     
+ Misses      40014    39985      -29     
+ Partials     5854     5850       -4     
Impacted Files Coverage Δ
models/unit.go 46.57% <0.00%> (-2.74%) ⬇️
modules/git/commit.go 56.47% <0.00%> (-0.72%) ⬇️
routers/repo/view.go 40.74% <0.00%> (-0.59%) ⬇️
models/error.go 39.84% <0.00%> (+0.31%) ⬆️
services/pull/pull.go 43.47% <0.00%> (+0.45%) ⬆️
models/gpg_key.go 54.07% <0.00%> (+0.56%) ⬆️
models/notification.go 66.07% <0.00%> (+0.88%) ⬆️
models/issue_comment.go 52.58% <0.00%> (+1.72%) ⬆️
modules/notification/notification.go 87.60% <0.00%> (+2.47%) ⬆️
modules/notification/base/null.go 78.94% <0.00%> (+2.63%) ⬆️
... and 2 more

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 1b3dbdb...bccfe7c. Read the comment docs.

@lunny lunny merged commit ea40eb7 into go-gitea:master Apr 24, 2021
@lunny
Copy link
Member

lunny commented Apr 24, 2021

Please send backport

@techknowlogick techknowlogick deleted the techknowlogick-patch-4 branch April 24, 2021 16:03
techknowlogick added a commit to techknowlogick/gitea that referenced this pull request Apr 24, 2021
…tea#15604)

go panics otherwise with `panic: interface conversion: error is git.ErrNotExist, not *git.ErrNotExist`, thanks to Codeberg/Andi for reporting this.

Co-authored-by: Lunny Xiao <[email protected]>
@techknowlogick techknowlogick added the backport/done All backports for this PR have been created label Apr 24, 2021
techknowlogick added a commit that referenced this pull request Apr 25, 2021
… (#15610)

go panics otherwise with `panic: interface conversion: error is git.ErrNotExist, not *git.ErrNotExist`, thanks to Codeberg/Andi for reporting this.

Co-authored-by: Lunny Xiao <[email protected]>

Co-authored-by: Lunny Xiao <[email protected]>
@go-gitea go-gitea locked and limited conversation to collaborators Jun 4, 2021
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.

5 participants