search

bagetter / BaGetter

A lightweight NuGet and symbol server
https://www.bagetter.com
MIT License
166 stars 38 forks source link

Enable nullability #96

Open Regenhardt opened 4 months ago

Regenhardt commented 4 months ago

Enabling nullability will make it easier to not accidentally pass something potentially uninitialized and also simpler to see at what point we don't have to null-check everything.

I guess immediately enabling it globally would make a mess so maybe start with enabling it in BaGetter.Protocol and putting question marks where needed, then BaGetter.Core, then look how far that branch goes without getting tired.

Just make a commit per project converted so we can merge early in order to minimize interference.

seriouz commented 3 months ago

I am pro on this. Because I personally like nullability very much, guarding us of Object reference not set to an instance of an object 😅. Walking through each project sounds very logically.

github-actions[bot] commented 4 weeks ago

This issue is stale because it has been open for 90 days with no activity. Remove the stale label, comment, or this will be closed in 5 days.

github-actions[bot] commented 3 weeks ago

This issue was closed because it has been stale for 5 days with no activity.