search

ipfs-shipyard / net-ipfs-core

MIT License
21 stars 8 forks source link

Enable nullable globally and fix issues #21

Closed erikmav closed 1 year ago

erikmav commented 1 year ago

For #19 . Remove some ArgumentNullException where nullability covers it to reduce runtime code (this is a change to public surface area, can be reverted - see changes in unit tests). Some opportunistic code cleanup ahead of #9 .

Arlodotexe commented 1 year ago

This will publish as soon as it merges, for use in https://github.com/ipfs-shipyard/net-ipfs-http-client/pull/21

erikmav commented 1 year ago

@Arlodotexe your review was reset so this never committed or published. When done I'll push a dependency update to https://github.com/ipfs-shipyard/net-ipfs-http-client/pull/21

Arlodotexe commented 1 year ago

Whoops! Sorry about that, let's kick it off.