-
-
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
Fix 404 when send pull request some situation #6871
Conversation
Co-Authored-By: lunny <xiaolunwen@gmail.com>
Co-Authored-By: lunny <xiaolunwen@gmail.com>
Co-Authored-By: lunny <xiaolunwen@gmail.com>
Co-Authored-By: lunny <xiaolunwen@gmail.com>
@lunny, would it be possible to write a document somewhere explaining what the permissions should be doing things? (This isn't the PR for that I know but I think it's a good example of one where we're not very clear about what permissions things should have.) |
Codecov Report
@@ Coverage Diff @@
## master #6871 +/- ##
=========================================
Coverage ? 41.33%
=========================================
Files ? 432
Lines ? 59532
Branches ? 0
=========================================
Hits ? 24607
Misses ? 31689
Partials ? 3236
Continue to review full report at Codecov.
|
@lunny are you able to backport? |
@zeripath I think that's a good idea. I will do that. |
With integration tests and should fix #6302