Skip to content
This repository was archived by the owner on Sep 11, 2020. It is now read-only.

repository: fix plain clone error handling regression #1028

Merged
merged 1 commit into from
Nov 27, 2018
Merged

repository: fix plain clone error handling regression #1028

merged 1 commit into from
Nov 27, 2018

Conversation

smola
Copy link
Collaborator

@smola smola commented Nov 27, 2018

PR #1008 introduced a regression by changing the errors returned by
PlainClone when a repository did not exist.

This change goes back to returned errors as they were in v4.7.0.

Fixes #1027

Signed-off-by: Santiago M. Mola [email protected]

@smola smola requested a review from mcuadros November 27, 2018 11:11
@smola smola changed the title repository: fix plain clone error handling regression [WIP] repository: fix plain clone error handling regression Nov 27, 2018
@smola smola removed the request for review from mcuadros November 27, 2018 11:21
PR #1008 introduced a regression by changing the errors returned by
PlainClone when a repository did not exist.

This change goes back to returned errors as they were in v4.7.0.

Fixes #1027

Signed-off-by: Santiago M. Mola <[email protected]>
@smola smola requested a review from mcuadros November 27, 2018 12:20
@smola smola changed the title [WIP] repository: fix plain clone error handling regression repository: fix plain clone error handling regression Nov 27, 2018
@mcuadros mcuadros merged commit 3dbfb89 into src-d:master Nov 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants