Uh oh!
There was an error while loading. Please reload this page.
Fix issue where PR without any description has its title edited - #79
Conversation
Mariatta
commented
Jan 24, 2018
Thanks @brettcannon for the quick review! |
brettcannon
commented
Jan 24, 2018
Looks like the version of @Mariatta Can you update the |
brettcannon
commented
Jan 24, 2018
I also just turned on pyup.io on this repo to help prevent this issue going forward. |
brettcannon
commented
Jan 24, 2018
Looks like pyup.io pushed #81 which should hopefully fix this. |
Mariatta
commented
Jan 24, 2018
Thanks! I'll rebase to get the latest changes. |
brettcannon
commented
Jan 24, 2018
#81 seems to do the trick, so probably just a merge off of |
This came up in python/cpython#5291 The PR did not have any description, and the title was edited to include the bpo number. It caused a 500 error, because bedevere did not anticipate this situation. Updated the code so that it takes into consideration when the body is empty.
ec9fd14 to
10e5cd5CompareCodecov Report
@@ Coverage Diff @@## master #79 +/- ##
=====================================
Coverage 100% 100% =====================================
Files 14 14 Lines 974 980 +6 Branches 56 56 =====================================
+ Hits 974 980 +6
Continue to review full report at Codecov.
|
This came up in python/cpython#5291
The PR did not have any description, and the title was
edited to include the bpo number.
It caused a 500 error, because bedevere did not anticipate
this situation.
Updated the code so that it takes into consideration when the
body is empty.