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
Following the discussion in #5 , I will limit the public surface of the API and add more comments for developers that want to dig further than the few methods in the Uuid static class.
make all generators internal as they are not meant to be used directly (or maybe just their constructors ?)
I noticed that the new method of the UuidTimestampGeneratorBase class has been changed to internal in version 3.0.0-beta, can it be modified back to public?
Because I support Sql Server and My SQL in the same app, and I injected the UuidTimestampGeneratorBase class into DI to generate IDs for different rules.
The reason that I'm making the generator classes internal is to prevent some misuse I've seen in some code base. These classes were not intended to be used outside of the library; it was a sloppy move from my part not to make them internal from the start.
For your use case, I recommend writing some code like this and using IUuidGenerator in the places where you used UuidTimestampGeneratorBase
Following the discussion in #5 , I will limit the public surface of the API and add more comments for developers that want to dig further than the few methods in the Uuid static class.
The text was updated successfully, but these errors were encountered: