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

fix a bug that caused auto ack'd messaages to be sent over and over #673

Merged
merged 1 commit into from
Jul 3, 2015

Conversation

jasoncalabrese
Copy link
Member

No description provided.

@jasoncalabrese jasoncalabrese added this to the f-release milestone Jul 3, 2015
@codecov-io
Copy link

Current coverage is 76.39%

Merging #673 into dev will change coverage by +0.10% by 7d4b813

Coverage Diff

@@              dev    #673   diff @@
=====================================
  Files          46      46       
  Stmts        2202    2215    +13
  Branches      414     415     +1
  Methods       302     305     +3
=====================================
+ Hit          1680    1692    +12
  Partial        12      12       
- Missed        510     511     +1

Uncovered Suggestions

  1. +0.59% via lib/treatments.js#18...30
  2. +0.54% via lib/utils.js#29...40
  3. +0.54% via lib/mqtt.js#157...168
  4. See 7 more...

Powered by Codecov

jasoncalabrese added a commit that referenced this pull request Jul 3, 2015
fix a bug that caused auto ack'd messaages to be sent over and over
@jasoncalabrese jasoncalabrese merged commit 3179a04 into dev Jul 3, 2015
@jasoncalabrese jasoncalabrese deleted the wip/autoack-once-fix branch July 3, 2015 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants