-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add error for fork already existing #7185
Conversation
Replaced DocURL with SwaggerURL Added error for fork already existing Signed-off-by: jolheiser <john.olheiser@gmail.com>
Signed-off-by: jolheiser <john.olheiser@gmail.com>
Signed-off-by: jolheiser <john.olheiser@gmail.com>
Signed-off-by: jolheiser <john.olheiser@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Except CI failed.
Codecov Report
@@ Coverage Diff @@
## master #7185 +/- ##
==========================================
- Coverage 41.53% 41.52% -0.02%
==========================================
Files 449 449
Lines 61302 61308 +6
==========================================
- Hits 25462 25458 -4
- Misses 32488 32495 +7
- Partials 3352 3355 +3
Continue to review full report at Codecov.
|
Partial fix for #7154 (considered partial because it addresses the error being misleading, however I think the issue author was asking for something else and this was just a byproduct)
The error returns the authed username, the repo they are trying to fork, and the name of the already existing fork (in case it was renamed)