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

Upgrade to .NET 5 #57

Open
ekolis opened this issue Dec 19, 2020 · 1 comment
Open

Upgrade to .NET 5 #57

ekolis opened this issue Dec 19, 2020 · 1 comment
Assignees
Labels
enhancement New feature or request technical debt Code that could be improved to improve maintainability.
Milestone

Comments

@ekolis
Copy link
Owner

ekolis commented Dec 19, 2020

Describe the feature you'd like to see
Upgrade this game from .NET Framework 4.x or whatever it's running under now to .NET 5.

Benefits
Not much benefit for players, might improve performance slightly if we're lucky, but it would mean we can use all the cool new C# 9 features like record types, nullable reference types, enhanced pattern matching, and so on.

Describe alternatives
It's .NET 5, not really many alternatives there besides maybe switching to .NET Core 3.1, but that's entirely supplanted by .NET 5 anyway. Mono or Unity, perhaps?

Additional context
One potential problem is, if I'm not mistaken WinForms apps targeting .NET 5 will not run in Mono, so we could be losing Linux/Mac compatibility until someone fixes that or a future version of .NET makes WinForms cross-platform.

@ekolis ekolis added the enhancement New feature or request label Dec 19, 2020
@ekolis ekolis added this to the 0.5.0 milestone Dec 19, 2020
@ekolis ekolis self-assigned this Jan 4, 2021
@ekolis ekolis pinned this issue Jan 4, 2021
@ekolis ekolis added the technical debt Code that could be improved to improve maintainability. label Jan 22, 2021
@ekolis
Copy link
Owner Author

ekolis commented Jan 22, 2021

I made some progress on this, but for some reason when I try to start a new game, the serializer crashes complaining that the mineral packet hull's "AllocatedComponent", whatever that is, is null...

@ekolis ekolis modified the milestones: 0.5.0, 0.4.3 Jan 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request technical debt Code that could be improved to improve maintainability.
Projects
None yet
Development

No branches or pull requests

1 participant