Skip to content

[microsoft-signalr] Add new port#10833

Merged
strega-nil merged 9 commits intomicrosoft:masterfrom
BrennanConroy:brecon/signalr
May 15, 2020
Merged

[microsoft-signalr] Add new port#10833
strega-nil merged 9 commits intomicrosoft:masterfrom
BrennanConroy:brecon/signalr

Conversation

@BrennanConroy
Copy link
Member

Describe the pull request

Adding the microsoft-signalr package to vcpkg.

@anurse @ras0219-msft

@LilyWangL LilyWangL changed the title Add microsoft-signalr package [microsoft-signalr] Add new port Apr 16, 2020
@LilyWangL LilyWangL assigned LilyWangL and unassigned PhoebeHui Apr 16, 2020
@BrennanConroy
Copy link
Member Author

@LilyWangL This should be good now? The Linux failure looks like an infrastructure issue.

@LilyWangL
Copy link
Contributor

LilyWangL commented May 6, 2020

@LilyWangL This should be good now? The Linux failure looks like an infrastructure issue.

Waiting for PR #11189 merge, this PR fix the CI baseline error.

@LilyWangL LilyWangL marked this pull request as ready for review May 12, 2020 09:26
@LilyWangL LilyWangL added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label May 12, 2020
Copy link
Contributor

@strega-nil strega-nil left a comment

Choose a reason for hiding this comment

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

Just a minor change here to avoid silent failures; thanks so much!

@strega-nil strega-nil added waiting for response and removed info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. labels May 12, 2020
@strega-nil
Copy link
Contributor

Awesome, thanks so much! Are you comfortable with me merging now?

@strega-nil strega-nil added info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. and removed waiting for response labels May 13, 2020
@BrennanConroy
Copy link
Member Author

Yes please :)

Thanks for helping clean this up!

@strega-nil
Copy link
Contributor

Alright, awesome! Thanks @BrennanConroy :)

@strega-nil strega-nil merged commit b5a2581 into microsoft:master May 15, 2020
@BrennanConroy BrennanConroy deleted the brecon/signalr branch May 15, 2020 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants