Skip to content

Conversation

@julianwachholz
Copy link

This allows setting additional arguments for AMQP queue declaration.
For example with RabbitMQ this will let you use Priority Queues by setting a x-max-priority on the queue and using x-priority headers on messages.

@codecov
Copy link

codecov bot commented Jun 19, 2019

Codecov Report

Merging #428 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #428      +/-   ##
=========================================
- Coverage   47.91%   47.9%   -0.02%     
=========================================
  Files          29      29              
  Lines        2642    2643       +1     
=========================================
  Hits         1266    1266              
- Misses       1237    1238       +1     
  Partials      139     139
Impacted Files Coverage Δ
v1/config/config.go 80% <ø> (ø) ⬆️
v1/tasks/signature.go 41.17% <ø> (ø) ⬆️
v1/brokers/amqp/amqp.go 4.16% <0%> (-0.02%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b05d372...aaf6436. Read the comment docs.

@julianwachholz
Copy link
Author

@RichardKnop I'm not sure how to add more coverage here? Is it okay to merge?

@RichardKnop RichardKnop merged commit 0d7c6f5 into RichardKnop:master Oct 8, 2019
@pmengcv
Copy link

pmengcv commented Dec 30, 2024

there has another problem
#825

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.

3 participants