Skip to content

Conversation

@kamronbatman
Copy link
Contributor

Removes double semicolons since they are not needed.

No specific testing since there are no functional changes. This is low risk.

@dnfclas
Copy link

dnfclas commented Nov 3, 2019

CLA assistant check
All CLA requirements met.

@Pilchie Pilchie added area-servers area-signalr Includes: SignalR clients and servers labels Nov 4, 2019
@Tratcher
Copy link
Member

Tratcher commented Nov 4, 2019

@Tratcher Tratcher self-assigned this Nov 4, 2019
@aspnet-hello
Copy link

This comment was made automatically. If there is a problem contact [email protected].

I've triaged the above build. I've created/commented on the following issue(s)
https://github.com/aspnet/AspNetCore-Internal/issues/3295

@kamronbatman
Copy link
Contributor Author

The triage doesn't exist?

@BrennanConroy
Copy link
Member

@kamronbatman don't worry about that, it's for our internal tracking!

The change looks good, I'll approve it and we'll merge it hopefully later today.

@halter73 halter73 merged commit d4f7a19 into dotnet:master Nov 5, 2019
@halter73
Copy link
Member

halter73 commented Nov 5, 2019

Thanks @kamronbatman!

@kamronbatman kamronbatman deleted the remove_double_semis branch November 6, 2019 07:19
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions area-signalr Includes: SignalR clients and servers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants