Some Cleanup #40

Merged
merged 1 commit into from Feb 13, 2017

Projects

None yet

5 participants

@lyrixx
Contributor
lyrixx commented Feb 9, 2017

No description provided.

@Oliboy50

thanks

@lucascourot

Thanks

src/AmqpBundle/Amqp/Consumer.php
- * @param string $deliveryTag Delivery tag of last message to nack.
- * @param integer $flags AMQP_NOPARAM or AMQP_REQUEUE to requeue the message(s).
+ * @param string $deliveryTag delivery tag of last message to nack
+ * @param int $flags aMQP_NOPARAM or AMQP_REQUEUE to requeue the message(s)
@lucascourot
lucascourot Feb 10, 2017 Member

"AMQP_NOPARAM" should stay uppercase

@lyrixx
lyrixx Feb 13, 2017 Contributor

Thanks Fixed

@jibe84
jibe84 approved these changes Feb 13, 2017 View changes
@jibe84 jibe84 merged commit 9642ff4 into M6Web:master Feb 13, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@lyrixx lyrixx deleted the lyrixx:my-2-cents branch Feb 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment