`check_for_update` should not happen on package load
prescod opened this issue · comments
check_for_update should not happen on module load. If I embed deepeval in my own codebase, it should not spit out messages or make networking calls on import.
check_for_update should be part of the CLI.
I completely agree.
At the very least, we need to have env override to disable this, similar to how telemetry can be disabled by setting DEEPEVAL_TELEMETRY_OPT_OUT to YES (has to be all capitals - it isn't parsed and is looking for exact string).