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

Use simpler csproj format #26

Merged
merged 1 commit into from
Apr 25, 2018
Merged

Conversation

twsouthwick
Copy link
Member

This project is fairly simple right now, but the older style csproj format that it is using can cause issues as a project grows especially concerning merge conflicts. As long as required development on VS 2017 is okay, this new format is much cleaner and simpler than the older project files.

@simonbuchan
Copy link

Wait, you can use the core csproj for full framework projects? Why isn't this default?

@twsouthwick
Copy link
Member Author

Because it doesn't work for all project types. Simple console style apps, though, work as expected. Others (like WebApps, WPF, etc), have issues at the moment (although the team is working to enable it for all). It is default, however, if you go through the ".NET Core -> Console App" menu and then change it to a whatever target framework (ie .NET 4.6) you want.

@simonbuchan
Copy link

Thanks! That makes sense. (I mean, except for the creating a netfx app via ".NET Core" 😉) Having looked at some of the msbuild .props and .targets now, I can see why it's taking time!

@miniksa
Copy link
Member

miniksa commented Apr 25, 2018

Sure this looks way simpler for a little tool like this.

@miniksa miniksa merged commit 4c3065f into microsoft:master Apr 25, 2018
@twsouthwick twsouthwick deleted the use-new-csproj branch April 25, 2018 21:27
carlos-zamora pushed a commit that referenced this pull request Aug 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants