Skip to content

Conversation

@eerhardt
Copy link
Member

System.Runtime.Serialization.Schema isn't getting a NuGet package created for it because it doesn't set IsPackable=true.

System.Runtime.Serialization.Schema isn't getting a NuGet package created for it because it doesn't set IsPackable=true.
@ghost ghost added the area-Serialization label Aug 24, 2022
@ghost ghost assigned eerhardt Aug 24, 2022
Copy link
Member

@StephenMolloy StephenMolloy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor

@carlossanlop carlossanlop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving, but I'd like @ViktorHofer or @ericstj to also please confirm that this is the expected thing to do for a new nuget package in main.

This will have to be backported to release/7.0-rc1 and release/7.0.

Copy link
Member

@ViktorHofer ViktorHofer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes this looks good.

@eerhardt
Copy link
Member Author

CI failures are unrelated.

@eerhardt eerhardt merged commit 3e7d596 into main Aug 25, 2022
@eerhardt eerhardt deleted the eerhardt-System.Runtime.Serialization.Schema-packable branch August 25, 2022 14:21
@carlossanlop
Copy link
Contributor

/backport to release/7.0-rc1

@github-actions
Copy link
Contributor

Started backporting to release/7.0-rc1: https://github.com/dotnet/runtime/actions/runs/2928410261

@ericstj
Copy link
Member

ericstj commented Aug 26, 2022

@dotnet/area-infrastructure-libraries this is the second instance of this problem in 7.0. We could avoid this by making folks set some property (or opt out) to define how they ship. I would expect every src project to be part of a shared framework, package, or explicitly set a property to indicate it isn’t shipping.

@ghost ghost locked as resolved and limited conversation to collaborators Sep 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants