You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In a PR elastio/elastio#6680 it became clear that because Config is marked non_exhaustive, the Config struct can't be initialized the normal way. Instead an ugly mut was used, so the Default() struct could be permuted with the appropriate config options.
This works fine but it's not particularly idiomatic or pleasant. A builder should be added for Config so that this type can be constructed without mutable state, while retaining the non_exhaustive attribute that is important for future flexibility.
The text was updated successfully, but these errors were encountered:
In a PR elastio/elastio#6680 it became clear that because
Config
is markednon_exhaustive
, theConfig
struct can't be initialized the normal way. Instead an uglymut
was used, so theDefault()
struct could be permuted with the appropriate config options.This works fine but it's not particularly idiomatic or pleasant. A builder should be added for
Config
so that this type can be constructed without mutable state, while retaining thenon_exhaustive
attribute that is important for future flexibility.The text was updated successfully, but these errors were encountered: