Skip to content
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.

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

Sync shared code from runtime #22619

Merged
1 commit merged into from
Jun 8, 2020
Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jun 6, 2020

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 June 6, 2020 08:07
@stephentoub
Copy link
Member

@Tratcher, as I was updating all of the runtime code for the new API, I forgot about the mirroring here. What's the best option for you? Should I revert the changes to this file in runtime and close this? Or just wait for this PR until aspnetcore picks up a newer runtime that has the new API?

@Tratcher
Copy link
Member

Tratcher commented Jun 8, 2020

No worries. I'll check tomorrow how long we expect it to take to get a runtime build. If it's just a few days then I might close the PRs until we get one that passes.

At least it's doing its job about nagging us 😁.

@Tratcher Tratcher self-assigned this Jun 8, 2020
@Tratcher Tratcher added the blocked The work on this issue is blocked due to some dependency label Jun 8, 2020
@Tratcher Tratcher closed this Jun 8, 2020
@Tratcher Tratcher reopened this Jun 8, 2020
@ghost
Copy link

ghost commented Jun 8, 2020

Hello @Tratcher!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 52fdd8f into master Jun 8, 2020
@ghost ghost deleted the github-action/sync-runtime-1591430846 branch June 8, 2020 20:17
@stephentoub
Copy link
Member

Cool, thanks for shepherding it through.

@dougbu dougbu removed the blocked The work on this issue is blocked due to some dependency label Oct 5, 2021
@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
This pull request was closed.
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

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