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
Since this only the first release in the 0.1 range, it might still be a good time to consider renaming this class. Event something like VaultlibClient would make it easier for IDE's and would prevent having to import-alias them. Especially since the "space" where this tool is used is already polluted with all kinds of Client classes.
The text was updated successfully, but these errors were encountered:
You don't think it being in the Psecio\Vaultlib namespace is enough separation? I get where you're coming from but I'm not a huge fan of including the name of the library in the class names. I'd rather them be named for what they are.
Since this only the first release in the 0.1 range, it might still be a good time to consider renaming this class. Event something like
VaultlibClient
would make it easier for IDE's and would prevent having to import-alias them. Especially since the "space" where this tool is used is already polluted with all kinds ofClient
classes.The text was updated successfully, but these errors were encountered: