Proper initialization - Githubissues

Cytraen / BungieSharper

A .NET client library for the Bungie.net API.

MIT License

15 stars 1 forks source link

Closed Cytraen closed 3 years ago

Cytraen commented 3 years ago

Currently considering

Internal IsInitialized bool, which will prevent any significant changes once set
Move solely to BungieClientConfig and remove other constructors altogether
Go back to / re-add {client}.SetX() methods

In any case, BungieClientConfig (available in cef8d4de4a19bb590c1d62db2625fe9eac2e9019 onwards) is not going anywhere

Cytraen commented 3 years ago

Going to add init-only setters which are only supported in .NET 5.0 onwards, so I'll be dropping support for .NET Standard 2.1 / Core 3.1 yet again.