Second 4.0 Changeset - Githubissues

hvanbakel / CsprojToVs2017

Tooling for converting pre 2017 project to the new Visual Studio 2017 format.

MIT License

1.08k stars 120 forks source link

Second 4.0 Changeset #227

Closed andrew-boyarshin closed 5 years ago

andrew-boyarshin commented 5 years ago

Improve .Console stub to guide user on the next steps
Drop "modern" cache for ConditionEvaluator
Use .NET Core 2.0+ Path.GetRelativePath routine (import CoreFX sources for .NET Standard 2.0, that is for .NET Framework)
Split Extensions into several files for maintainability
Add UpgradeUseComVisibleDefaultTransformation
Add UpgradeUseDefaultOutputPathTransformation
Fix CLI issues, improve help messages
Experimental Entity Framework detection (fix #220)
Project.AppendTargetFrameworkToOutputPath is now bool? instead of bool

P.S. It appears Unix support is now on par with Windows. Verified on a couple of open-source projects. 🎉

andrew-boyarshin commented 5 years ago

@hvanbakel I'm not sure how to do that. I mean, Travis-CI setup would be good (or Microsoft Azure, former VS Team Services). But that still won't cover all of it as we don't have enough tests.

andrew-boyarshin commented 5 years ago

Before 4.0 release we would need to update documentation (README) to recommend using new Interactive Wizard and evaluate-migrate cycle only for script usage or very advanced users.

andrew-boyarshin commented 5 years ago

@hvanbakel I would like you to take an additional look at README.md changes. If they seem ok to you, I'll merge this into master and update revision to 4.0.0-rc1.

hvanbakel commented 5 years ago

Nice!

On Sat, Jan 26, 2019, 06:04 Andrew Boyarshin <notifications@github.com wrote:

Merged #227 https://github.com/hvanbakel/CsprojToVs2017/pull/227 into master.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/hvanbakel/CsprojToVs2017/pull/227#event-2098856736, or mute the thread https://github.com/notifications/unsubscribe-auth/AD8FPAtoqH8PYaceLYNGhL2XmSvQIuxaks5vHGBkgaJpZM4aDQ87 .