Skip to content

Commit 79529a7

Browse files
committed
Revert "expose write_legacy"
This reverts commit af1366f.
1 parent af1366f commit 79529a7

File tree

1 file changed

+4
-11
lines changed

1 file changed

+4
-11
lines changed

src/iterative_telemetry/__init__.py

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -37,15 +37,13 @@ def __init__(
3737
url=URL,
3838
token=TOKEN,
3939
debug: bool = False,
40-
write_legacy: bool = True,
4140
):
4241
self.tool_name = tool_name
4342
self.tool_version = tool_version
4443
self.enabled = enabled
4544
self.url = url
4645
self.token = token
4746
self.debug = debug
48-
self.write_legacy = write_legacy
4947
if self.debug:
5048
logger.setLevel(logging.DEBUG)
5149
logger.debug("IterativeTelemetryLogger is in debug mode")
@@ -77,8 +75,7 @@ def is_enabled(self):
7775
return (
7876
os.environ.get(DO_NOT_TRACK_ENV, None) is None and self.enabled()
7977
if callable(self.enabled)
80-
else self.enabled
81-
and _find_or_create_user_id(self.write_legacy) is not None
78+
else self.enabled and _find_or_create_user_id() is not None
8279
)
8380

8481
def send(
@@ -156,7 +153,7 @@ def _runtime_info(self):
156153
# "tool_source": self.tool_source, # TODO
157154
# "scm_class": _scm_in_use(),
158155
**_system_info(),
159-
"user_id": _find_or_create_user_id(self.write_legacy),
156+
"user_id": _find_or_create_user_id(),
160157
"group_id": "", # TODO
161158
}
162159

@@ -196,7 +193,7 @@ def generate_id():
196193

197194

198195
@lru_cache(None)
199-
def _find_or_create_user_id(write_legacy=True):
196+
def _find_or_create_user_id():
200197
"""
201198
The user's ID is stored on a file under the global config directory.
202199
The file should contain JSON with a `user_id` key:
@@ -224,11 +221,7 @@ def _find_or_create_user_id(write_legacy=True):
224221

225222
# only for non-DVC packages,
226223
# write legacy file in case legacy DVC is installed later
227-
if (
228-
write_legacy
229-
and not old.exists()
230-
and uid.lower() != DO_NOT_TRACK_VALUE.lower()
231-
):
224+
if not old.exists() and uid.lower() != DO_NOT_TRACK_VALUE.lower():
232225
json.dump({"user_id": uid}, old.open("w", encoding="utf8"))
233226

234227
if uid.lower() != DO_NOT_TRACK_VALUE.lower():

0 commit comments

Comments
 (0)