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.

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

added Buffered overloads of GetSubscriber, GetPublisher to GlobalMessagePipe #83

Merged
merged 3 commits into from
Jun 23, 2022

Conversation

AGeorgy
Copy link
Contributor

@AGeorgy AGeorgy commented May 30, 2022

No description provided.

@AGeorgy AGeorgy closed this Jun 10, 2022
@NinjaQuinn
Copy link

@AGeorgy I actually could use this, why was it closed instead of merged?

@quinnhoener
Copy link

@neuecc , wondering your thoughts on getting something like this merged to the main product? I'm not using a 3rd party DI system and want to use buffered messages with the GlobalMessagePipe.

Thanks!

@neuecc
Copy link
Member

neuecc commented Jun 22, 2022

Yes, ok to merge.
Shall I Reopen and Merge?

@quinnhoener
Copy link

quinnhoener commented Jun 22, 2022

@neuecc yes please! I couldn't figure out how to do that (maybe I don't have access for that).

Thank you!

@quinnhoener
Copy link

Just heads up it looks like this PR might be manually setting the package.json version number. I noticed git history looks like there's a GitHub Action tied to incrementing that number across all the packages?

@AGeorgy AGeorgy reopened this Jun 22, 2022
@neuecc
Copy link
Member

neuecc commented Jun 23, 2022

For now, let's get it out with 1.7.4.

@neuecc neuecc merged commit 0a69940 into Cysharp:master Jun 23, 2022
@quinnhoener
Copy link

Thanks for this! Greatly appreciate the quick turnaround.

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.

4 participants