Skip to content

Update MessagePack-CSharp #30608

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
Merged
merged 1 commit into from
Mar 10, 2021
Merged

Update MessagePack-CSharp #30608

merged 1 commit into from
Mar 10, 2021

Conversation

@pranavkm
Copy link
Contributor

@pranavkm pranavkm commented Mar 3, 2021

No description provided.

@pranavkm pranavkm requested a review from dougbu March 3, 2021 19:36
@dougbu
Copy link
Contributor

dougbu commented Mar 3, 2021

Ta for this and #30607 @pranavkm❕

@pranavkm
Copy link
Contributor Author

pranavkm commented Mar 3, 2021

FYI @blowdart

@Pilchie Pilchie added area-blazor Includes: Blazor, Razor Components area-signalr Includes: SignalR clients and servers labels Mar 4, 2021
@pranavkm
Copy link
Contributor Author

pranavkm commented Mar 9, 2021

@wtgodbe there's also this one.

@dougbu
Copy link
Contributor

dougbu commented Mar 10, 2021

@Pilchie can we update the MessagePack submodule as tell-mode❔ @pranavkm is there much in the span between old and new that's not pure infrastructure❔

@dougbu dougbu added this to the 5.0.5 milestone Mar 10, 2021
@wtgodbe
Copy link
Member

wtgodbe commented Mar 10, 2021

I already merged the 3.1 version of this as tell-mode, this one should be good too

@wtgodbe wtgodbe added the tell-mode Indicates a PR which is being merged during tell-mode label Mar 10, 2021
@wtgodbe wtgodbe merged commit f3c9c53 into release/5.0 Mar 10, 2021
@wtgodbe wtgodbe deleted the prkrishn/messagepack-csharp-50 branch March 10, 2021 17:55
@pranavkm
Copy link
Contributor Author

pranavkm commented Mar 10, 2021

This is all infstracture. New commit = old commit + changes to nuget.config.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@dougbu dougbu dougbu approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

area-blazor Includes: Blazor, Razor Components area-signalr Includes: SignalR clients and servers tell-mode Indicates a PR which is being merged during tell-mode

Projects

None yet

Milestone

5.0.5

Development

Successfully merging this pull request may close these issues.

5 participants

@pranavkm @dougbu @wtgodbe @Pilchie