Fixed the deliveryMode bug. #1594

Merged
merged 1 commit into from Sep 14, 2016

Conversation

Projects
None yet
3 participants
@thihara
Contributor

thihara commented Sep 14, 2016

Fixes a bug rendering rabbitmq plugin unusable.

Changes proposed in this pull request:

  • Define the deliveryMode variable globally.
  • Actually use the deliveryMode variable.

Checklist:

  • docs updated
  • tests updated

The variable wasn't defined globally. It wasn't being used when initializing the queue.

Fixed the deliveryMode bug.
The variable wasn't defined globally. It wasn't being used when initializing the queue.
@codecov-io

This comment has been minimized.

Show comment
Hide comment
@codecov-io

codecov-io Sep 14, 2016

Current coverage is 35.93% (diff: 100%)

Merging #1594 into master will not change coverage

@@             master      #1594   diff @@
==========================================
  Files            26         26          
  Lines          6521       6521          
  Methods         847        847          
  Messages          0          0          
  Branches       1698       1698          
==========================================
  Hits           2343       2343          
  Misses         4178       4178          
  Partials          0          0          

Powered by Codecov. Last update 7632d69...a368e25

codecov-io commented Sep 14, 2016

Current coverage is 35.93% (diff: 100%)

Merging #1594 into master will not change coverage

@@             master      #1594   diff @@
==========================================
  Files            26         26          
  Lines          6521       6521          
  Methods         847        847          
  Messages          0          0          
  Branches       1698       1698          
==========================================
  Hits           2343       2343          
  Misses         4178       4178          
  Partials          0          0          

Powered by Codecov. Last update 7632d69...a368e25

@baudehlo baudehlo merged commit ecc78a6 into haraka:master Sep 14, 2016

2 checks passed

codecov/project 35.93% (+0.00%) compared to 7632d69
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@baudehlo

This comment has been minimized.

Show comment
Hide comment
@baudehlo

baudehlo Sep 14, 2016

Collaborator

Merged, thanks.

Collaborator

baudehlo commented Sep 14, 2016

Merged, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment