Conversation
| { | ||
| case (int)HttpStatusCode.Continue: | ||
| return BytesStatus100; | ||
| return "100"u8; |
There was a problem hiding this comment.
What the devil is this? A new C# feature for representing UTF8 bytes?
There seem to be errors when building here. We might not have a recent enough SDK.
There was a problem hiding this comment.
There was a problem hiding this comment.
Suspect we're now much closer to an SDK or a compiler that can handle this. Suggest updating to the LKG SDK in this PR if you can't wait 'til Monday for our usual SDK bump.
|
Blocked by #41158 |
Tratcher
added
the
blocked
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
Oops, that might be a wasted run given failure of what @BrennanConroy started |
|
Is the compiler updated with the SDK? A new C# compiler is required for the new syntax in this sync. |
|
@BrennanConroy and @sebastienros (?) are already looking at an SDK update, but there are other challenges taking the latest SDK. |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
ghost
added this to the
amcasey
added
area-networking
5 participants
This PR was automatically generated to sync shared code changes from runtime. Fixes #18943