Skip to content

added Buffered overloads of GetSubscriber, GetPublisher to GlobalMessagePipe#83

Merged
neuecc merged 3 commits intoCysharp:masterfrom
AGeorgy:master
Jun 23, 2022
Merged

added Buffered overloads of GetSubscriber, GetPublisher to GlobalMessagePipe#83
neuecc merged 3 commits intoCysharp:masterfrom
AGeorgy:master

Conversation

@AGeorgy
Copy link

@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