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

Switch to the new csproj system #315

Closed
SeleDreams opened this issue Dec 18, 2019 · 1 comment · Fixed by godotengine/godot#40595
Closed

Switch to the new csproj system #315

SeleDreams opened this issue Dec 18, 2019 · 1 comment · Fixed by godotengine/godot#40595

Comments

@SeleDreams
Copy link

Describe the project you are working on:
I am working on an RPG in C# using Godot Mono 3.1.2.

Describe the problem or limitation you are having in your project:
I don't have a limitation in particular, however, this suggestion should make it easier to work with vs code etc to work on C# projects by making the syntax of the csproj more human-like.

Describe how this feature / enhancement will help you overcome this problem or limitation:
By switching to the new csproj syntax used by net standard and net core projects, it should be easier to edit information from the solution without being lost with the amount of useless details contained in the old autogenerated csproj files

Show a mock up screenshots/video or a flow diagram explaining how your proposal will work:
instead of using this clunky kind of csproj
image

godot could generate csproj files like this
image

it would only require to use net45 in TargetFramework without the netstandard2.1. This format is compatible with mono and visual studio post 2017, so it should work without issues.

@neikeq
Copy link

neikeq commented Dec 18, 2019

I plan to switch both to project SDKs and to .NET Standard 2.1 sometime Q1 2020.
We will use our on SDK on top of Microsoft.NET.Sdk.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants