Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Sync shared code from runtime #39453

Merged
merged 1 commit into from
Jan 12, 2022
Merged

Conversation

github-actions[bot]
Copy link
Contributor

This PR was automatically generated to sync shared code changes from runtime. Fixes #18943

@github-actions github-actions bot requested a review from a team January 12, 2022 08:04
@Tratcher Tratcher requested a review from pranavkm January 12, 2022 16:18
@Tratcher
Copy link
Member

Pranav, it looks like this should be reverted and excluded. These methods aren't intend to be static, they're just not fully implemented yet.

@pranavkm
Copy link
Contributor

The rule is suppressed for these types: https://github.com/dotnet/aspnetcore/blob/main/.editorconfig#L285, but I think the codefix might have been aggressive about updating this file. We should be able to merge this without any additional changes.

@Tratcher Tratcher enabled auto-merge (squash) January 12, 2022 18:20
@Tratcher Tratcher merged commit d381765 into main Jan 12, 2022
@Tratcher Tratcher deleted the github-action/sync-runtime-1641974651 branch January 12, 2022 22:55
@ghost ghost added this to the 7.0-preview1 milestone Jan 12, 2022
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Aug 24, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The runtime<->aspnetcore shared src is out of sync
3 participants