Skip to content

Conversation

zeripath
Copy link
Contributor

Unfortunately #16435 asserts the wrong error and should use
os.LinkError not os.PathError.

Fix #16439

Unfortunately go-gitea#16435 asserts the wrong error and should use
os.LinkError not os.PathError.

Fix go-gitea#16439

Signed-off-by: Andrew Thornton <[email protected]>
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jul 16, 2021
zeripath added a commit to zeripath/gitea that referenced this pull request Jul 16, 2021
Backport go-gitea#16461

Unfortunately go-gitea#16435 asserts the wrong error and should use
os.LinkError not os.PathError.

Fix go-gitea#16439

Signed-off-by: Andrew Thornton <[email protected]>
zeripath added a commit to zeripath/gitea that referenced this pull request Jul 16, 2021
Backport go-gitea#16461

Unfortunately go-gitea#16435 asserts the wrong error and should use
os.LinkError not os.PathError.

Fix go-gitea#16439

Signed-off-by: Andrew Thornton <[email protected]>
@codecov-commenter
Copy link

Codecov Report

Merging #16461 (fc01ed8) into main (95f4004) will increase coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #16461      +/-   ##
==========================================
+ Coverage   45.42%   45.44%   +0.01%     
==========================================
  Files         717      717              
  Lines       84182    84182              
==========================================
+ Hits        38243    38255      +12     
+ Misses      39789    39780       -9     
+ Partials     6150     6147       -3     
Impacted Files Coverage Δ
modules/util/remove.go 58.53% <0.00%> (ø)
services/pull/pull.go 41.93% <0.00%> (-0.44%) ⬇️
modules/git/blame.go 67.14% <0.00%> (+1.42%) ⬆️
modules/process/manager.go 75.30% <0.00%> (+2.46%) ⬆️
modules/queue/workerpool.go 57.25% <0.00%> (+4.19%) ⬆️

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 95f4004...fc01ed8. Read the comment docs.

@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 Jul 16, 2021
zeripath added a commit that referenced this pull request Jul 16, 2021
Backport #16461

Unfortunately #16435 asserts the wrong error and should use
os.LinkError not os.PathError.

Fix #16439

Signed-off-by: Andrew Thornton <[email protected]>
@6543 6543 merged commit b08e14b into go-gitea:main Jul 16, 2021
techknowlogick pushed a commit that referenced this pull request Jul 16, 2021
Backport #16461

Unfortunately #16435 asserts the wrong error and should use
os.LinkError not os.PathError.

Fix #16439

Signed-off-by: Andrew Thornton <[email protected]>
@zeripath zeripath deleted the fix-16439-os.LinkError-not-PathError branch July 16, 2021 17:23
@zeripath zeripath added the backport/done All backports for this PR have been created label Jul 21, 2021
AbdulrhmnGhanem pushed a commit to kitspace/gitea that referenced this pull request Aug 10, 2021
Unfortunately go-gitea#16435 asserts the wrong error and should use
os.LinkError not os.PathError.

Fix go-gitea#16439

Signed-off-by: Andrew Thornton <[email protected]>
@go-gitea go-gitea locked and limited conversation to collaborators Oct 19, 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.

6 participants