achilleasa / dart_amqp

Dart AMQP client implementing protocol version 0.9.1
MIT License
79 stars 40 forks source link

Expose deliveryTag from AmqpMessage #80

Closed brizaldi closed 2 years ago

brizaldi commented 2 years ago

Allow getting deliveryTag directly from message.

consumer.listen((AmqpMessage message) {
  print(message.deliveryTag);
});

Fixes #79

brizaldi commented 2 years ago

relevant issue #79

achilleasa commented 2 years ago

Thanks for your contribution.

Please rebase your changes on top of HEAD so we can get a clean CI run and get this merged.

brizaldi commented 2 years ago

Hi @achilleasa the CI still failed, any suggestions?

achilleasa commented 2 years ago

Hi @brizaldi. I think you haven't rebased your changes on top of the current master branch. If you do, you will get the CI fix from https://github.com/achilleasa/dart_amqp/commit/f1844ca21afdd0f95a7af8ea37fb1995ca78986c.

coveralls commented 2 years ago

Coverage Status

Coverage decreased (-0.1%) to 93.01% when pulling aa0259ebec002c695bbf847d2cc8189ea3c86b1b on brizaldi:expose-delivery-tag into 9a76e466e9ca6caaba93c5a64d5d3a7ee2247a4b on achilleasa:master.