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

Enable Nullable reference types #310

Open
hahn-kev opened this issue Aug 17, 2024 · 0 comments
Open

Enable Nullable reference types #310

hahn-kev opened this issue Aug 17, 2024 · 0 comments

Comments

@hahn-kev
Copy link
Contributor

hahn-kev commented Aug 17, 2024

Since starting to use LCM with Fieldworks lite I'm running into places where NRT would help avoid runtime bugs.

Read about NRT here. Migration guide.

We need to plan how to roll this out, one huge PR is going to make merging difficult so it should be done in stages. Maybe we do it per project or maybe at a more fine grained level.

Generated code will probably be the biggest area that will be tricky to figure out.

I'm not sure how this works with dotnet framework, some things I read said it should work but it requires testing.

Finally we should consider if we want to make null issues errors or warnings. For Lexbox we found that by making them errors we actually paid attention and we largely ignored it when it was just a warning.

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

No branches or pull requests

1 participant