Skip to content
New issue

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

Sign up for GitHub

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

Jump to bottom

Added demo for amqp. #2

Merged
merged 4 commits into from
Jun 5, 2020
Merged

Added demo for amqp. #2

merged 4 commits into from
Jun 5, 2020

Conversation

limingxinleo
Copy link
Member

@limingxinleo limingxinleo commented May 17, 2020 •
edited
Loading

fix #1

@limingxinleo limingxinleo requested review from huangzhhui and Reasno May 17, 2020 03:19
huangzhhui
huangzhhui reviewed May 17, 2020
View reviewed changes
README-CN.md Outdated

require_once __DIR__ . '/vendor/autoload.php';

class DemoConsumer extends Amqp\Message\ConsumerMessage
Copy link
Member

Choose a reason for hiding this comment

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

这里这么写感觉有点乱，改成匿名类的写法会不会好一些呢？

Copy link
Member Author

Choose a reason for hiding this comment

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

嗯，我今天再改改

Reasno
Reasno approved these changes May 19, 2020
View reviewed changes
README-CN.md Outdated

$app->addProcess(function () use ($container) {
$message = new DemoConsumer();
$consumer = $container->get(Amqp\Consumer::class);
Copy link
Member

Choose a reason for hiding this comment

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

这里直接用$this就行

@limingxinleo limingxinleo merged commit 4c419d9 into master Jun 5, 2020
@limingxinleo limingxinleo deleted the v1.0-limx-amqp-doc branch June 5, 2020 10:26
@limFun limFun mentioned this pull request May 17, 2023
Open
@drigolin drigolin mentioned this pull request Oct 21, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@huangzhhui huangzhhui huangzhhui left review comments

@Reasno Reasno Reasno approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

基于nano 实现amqp 消息投递和消费
3 participants
@limingxinleo @Reasno @huangzhhui