Skip to content
This repository has been archived by the owner on Jun 2, 2024. It is now read-only.

fix: limit sync fails email notice #1006

Merged
merged 1 commit into from
Aug 2, 2016
Merged

fix: limit sync fails email notice #1006

merged 1 commit into from
Aug 2, 2016

Conversation

fengmk2
Copy link
Member

@fengmk2 fengmk2 commented Aug 2, 2016

and skip JSONResponseFormatError and 504 server error

and skip JSONResponseFormatError and 504 server error
@fengmk2 fengmk2 added this to the 2.x milestone Aug 2, 2016
@mention-bot
Copy link

@fengmk2, thanks for your PR! By analyzing the annotation information on this pull request, we identified @dead-horse to be a potential reviewer

@fengmk2
Copy link
Member Author

fengmk2 commented Aug 2, 2016

现在每天美国时间高峰期,经常会抛错,而且这些错误都是可以忽略的。

@fengmk2
Copy link
Member Author

fengmk2 commented Aug 2, 2016

image

@codecov-io
Copy link

codecov-io commented Aug 2, 2016

Current coverage is 88.62% (diff: 100%)

Merging #1006 into master will increase coverage by 0.15%

@@             master      #1006   diff @@
==========================================
  Files            85         85          
  Lines          3314       3314          
  Methods         340        340          
  Messages          0          0          
  Branches        618        618          
==========================================
+ Hits           2932       2937     +5   
+ Misses          382        377     -5   
  Partials          0          0          

Sunburst

Powered by Codecov. Last update bfbfef8...dcdb27c

@fengmk2 fengmk2 merged commit 7c2f947 into master Aug 2, 2016
@fengmk2 fengmk2 deleted the limit-error-message branch August 2, 2016 13:42
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.

4 participants