search

DevChatter / ChatterBot

Highly modular Twitch chat bot.
MIT License
10 stars 5 forks source link

Switch to c# 8 with nullable reference types #34

Closed SimonGeering closed 4 years ago

SimonGeering commented 4 years ago

Is your feature request related to a problem? Please describe. The root cause of #33 is potentially a null reference exception.

Describe the solution you'd like To prevent null reference exceptions we should switch all projects to use c#8 language features with nullable reference types:

    <LangVersion>8.0</LangVersion>
    <nullable>enable</nullable>

Describe alternatives you've considered

Additional context I have disabled unit test that are causing this issue for now in the PR for #9

SimonGeering commented 4 years ago

Happy to do a PR for this if you want to assign it to me @benrick .