Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

[release/3.1] Make PathAssemblyResolver handle Retargetable #42768

Merged
merged 3 commits into from
Jan 14, 2020

Conversation

ericstj
Copy link
Member

@ericstj ericstj commented Dec 10, 2019

Release/3.1 port of fca6032 + package authoring.

@ericstj
Copy link
Member Author

ericstj commented Dec 10, 2019

Fixes dotnet/runtime#730
/cc @mmitche

@ericstj ericstj added the Servicing-approved Approved for servicing release label Dec 10, 2019
@ericstj ericstj added Servicing-consider Issue for next servicing release review and removed Servicing-consider Issue for next servicing release review labels Dec 10, 2019
@tmat

This comment has been minimized.

@ericstj
Copy link
Member Author

ericstj commented Dec 10, 2019

Failure here appears to be related to how we support netcoreapp3.1 package tests. We didn't catch it in the 3.1 SDK update PR since we weren't building packages at that time. I'll fix this in the package testing infra.

@mmitche
Copy link
Member

mmitche commented Dec 11, 2019

Hold this until 3.1.2, as I think the other fix is slipping too, and aspnetcore is just about ready to go.

@mmitche mmitche added the * NO MERGE * The PR is not ready for merge yet (see discussion for detailed reasons) label Dec 11, 2019
@ericstj
Copy link
Member Author

ericstj commented Jan 6, 2020

@mmitche where are we at for 3.1.2?

@mmitche
Copy link
Member

mmitche commented Jan 6, 2020

Not started yet. Waiting on the last signoffs for 3.1.1

@danmoseley danmoseley modified the milestones: 3.1.x, 3.1.2 Jan 9, 2020
@danmoseley
Copy link
Member

@mmitche @ericstj we can remove "no-merge" right? and merge this now.

@mmitche
Copy link
Member

mmitche commented Jan 14, 2020

Not until branding is updated.

@danmoseley
Copy link
Member

Sure, I meant we can remove "no-merge" ?

@Anipik Anipik removed the * NO MERGE * The PR is not ready for merge yet (see discussion for detailed reasons) label Jan 14, 2020
@Anipik Anipik merged commit e021226 into dotnet:release/3.1 Jan 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Reflection Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants