Skip to content
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

1. Change response processing to copy out of a channel to array #30

Merged
merged 7 commits into from
Sep 29, 2016

Conversation

ekaley
Copy link

@ekaley ekaley commented Sep 27, 2016

  1. re-write processing logic
  2. wrap error handling & handle response errors from w/in go routines

2. re-write processing logic
3. wrap error handling & handle response errors from w/in go routines
@coveralls
Copy link

Coverage Status

Coverage increased (+8.2%) to 100.0% when pulling e1b2e89 on concurrency_fix into 4b31f10 on master.

@coveralls
Copy link

coveralls commented Sep 28, 2016

Coverage Status

Coverage remained the same at 91.795% when pulling d2fbcf6 on concurrency_fix into 4b31f10 on master.

@coveralls
Copy link

coveralls commented Sep 28, 2016

Coverage Status

Coverage remained the same at 91.795% when pulling 9fe3d14 on concurrency_fix into 4b31f10 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+8.2%) to 100.0% when pulling ab0ea85 on concurrency_fix into 4b31f10 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.5%) to 90.303% when pulling f8cef86 on concurrency_fix into 4b31f10 on master.

make an error msg print in json
@coveralls
Copy link

coveralls commented Sep 28, 2016

Coverage Status

Coverage remained the same at 91.795% when pulling 6e1a508 on concurrency_fix into 4b31f10 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.5%) to 90.303% when pulling 7788709 on concurrency_fix into 4b31f10 on master.

@king-jam king-jam merged commit c59534e into master Sep 29, 2016
@king-jam king-jam deleted the concurrency_fix branch September 29, 2016 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants