CloudNimble / PackageReferenceUpgrader

A VS2017 Extension that helps legacy apps migrate off of packages.config.
MIT License
28 stars 9 forks source link

.net compiler imports removed #4

Closed PhilPJL closed 6 years ago

PhilPJL commented 6 years ago

Installed product versions

Description

The .net compiler import statements are removed - not sure if an issue or not.

snip

robertmclaws commented 6 years ago

This is as designed, the PackageReferences are supposed resolve those automatically. Are you seeing an issue because the imports are missing?

PhilPJL commented 6 years ago

Roberts, as I said - not sure if an issue or not. I'll have to check the changes in and kick off a build then see what happens - will do later.

robertmclaws commented 6 years ago

Yeah, this is by design. The PackageReference system handles .targets in the packages automatically.

I'm closing this one for now, but please feel free to re-open if you notice that it causes an issue I haven't encountered yet.