Skip to content

Upgrade php-amqplib #4

New issue

Have a question about this project? No Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “No 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? No Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

masterjus
Copy link

Yesterday php-amqplib was upgraded to php8 with v3.0.0

@@ -68,15 +70,13 @@ public function testOnAcknowledgeShouldAcknowledgeMessage()
$channel
->expects($this->once())
->method('basic_ack')
->with(167)
;
->with(167);
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

autoformatter did this, I could change it back if necessary

@liasica
Copy link

liasica commented Mar 17, 2021

thanks a lot.

@makasim
Copy link
Member

makasim commented Mar 17, 2021

Thanks for your pull request! We love contributions.

However, this repository is what we call a "subtree split": a read-only copy of one directory of the main enqueue-dev repository. It is used by Composer to allow developers to depend on specific Enqueue package.

If you want to contribute, you should instead open a pull request on the main repository:

https://github.com/php-enqueue/enqueue-dev

Read the contribution guide

https://github.com/php-enqueue/enqueue-dev/blob/master/docs/contribution.md

Thank you for your contribution!

@makasim makasim closed this Mar 17, 2021
@masterjus masterjus deleted the upgrade-php-amqplib branch March 17, 2021 14:08
@masterjus masterjus restored the upgrade-php-amqplib branch March 17, 2021 14:24
@masterjus
Copy link
Author

It's done php-enqueue/enqueue-dev#1146

No Sign up for free to join this conversation on GitHub. Already have an account? No Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants